Don't include full paths in module names in error messages
See original GitHub issueTypeScript Version: 4.0.5 (also tested with typescript@next
)
Search Terms: namespace has no exported member error
Description
Currently certain error messages shows full file paths. For example:
Namespace ‘“y:/projects/opf/portal/OPF.Portal.Web/Content/Scripts/DinSide/App/Store/index”’ has no exported member ‘All’. ts(2694)
Could not find a declaration file for module ‘shortid’. ‘y:/projects/opf/portal/OPF.Portal.Web/node_modules/shortid/index.js’ implicitly has an ‘any’ type. Try
npm install @types/shortid
if it exists or add a new declaration (.d.ts) file containingdeclare module 'shortid';
ts(7016)
Our project was started without strict: true
(and few @typescript-eslint
rules), so the code base includes several thousand errors. To be able to gradually fix them, we have started to use betterer
, which includes committing a snapshot of all the current errors. Snapshot is then used to report report when we fix errors, and fail our build if we add new errors. But because of certain Typescript errors using full file paths, we’re getting unnecessary diffs in this snapshot.
Would it be possible to use file paths relative to the project/tsconfig.json root in the error messages instead?
So, with tsconfig.json in y:/projects/opf/portal/OPF.Portal.Web/
, the error message would instead be:
Namespace ‘“Content/Scripts/DinSide/App/Store/index”’ has no exported member ‘All’. ts(2694)
An alternative solution could be to change betterer
instead, make it auto-replace file paths with relative paths, or something like that. But figured I could at least ask her, especially since it could maybe be an issue for other tools as well.
Related Issues: Couldn’t find any
Issue Analytics
- State:
- Created 3 years ago
- Comments:7 (3 by maintainers)
It’s any time we reference the name of a module, which are internally represented by their full disk path. During error message construction we should convert them back to tsconfig-relative paths
👍