pzavolinsky / ts-unused-exports

ts-unused-exports finds unused exported symbols in your Typescript project
MIT License
749 stars 49 forks source link

Fix/issue support tsconfig extends #202

Closed mrseanryan closed 2 years ago

mrseanryan commented 3 years ago

Adds support for "extends" property in the tsconfig.json, so the relevant properties are read from base configs as appropriate (via TypeScript).

Internal: uses absolute paths which is less ambiguous in case a project has different types with same local paths.


A continuation of #200 (mainly due to Travis issue)

Possibly would solve #102


ref = https://github.com/microsoft/TypeScript/issues/18865 ref = https://npm.io/search/keyword:tsconfig

mrseanryan commented 3 years ago

update:

TECH DEBT: we don't read baseUrl from the result - so probably that is also a bug?

mrseanryan commented 3 years ago
boredland commented 3 years ago

really nice you followed up with this!

mrseanryan commented 3 years ago

@boredland this PR supports baseUrl reading, even when 'extends' is in use. If you have time to try it out, that would be great!

@pzavolinsky this is a major change, but I believe a good improvement.

Internally it is a bit simpler (just 'baseUrl' - baseDir and rootDir are no more - I always found it hard to differentiate them anyway....) It should solve some issues for projects that use base tsconfig files (via 'extends').

mrseanryan commented 3 years ago

@pzavolinsky it seems a bit difficult to get PRs merged such as this one...

Perhaps automating the build could help - ref #208 or #48

andrewbihl commented 2 years ago

What's the status here? I'd like to use this without forking

mrseanryan commented 2 years ago

Only @pzavolinsky has access to fix the build + release to npm.

pzavolinsky commented 2 years ago

Sorry I've been out for so long. We need to do some clean up and fix CI, in the meantime, @mrseanryan if you can merge latest to master, I'll release manually.

mrseanryan commented 2 years ago

@pzavolinsky its merged! thank you

pzavolinsky commented 2 years ago

Released as 8.0.0, sorry for the long wait!