Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for project: true or useProjectService: true ? #282

Open
Samuel-Therrien-Beslogic opened this issue May 1, 2024 · 4 comments
Open

Comments

@Samuel-Therrien-Beslogic
Copy link

Samuel-Therrien-Beslogic commented May 1, 2024

Hi! I'm wondering if the resolver currently or plans on supporting project: true like https://typescript-eslint.io/packages/parser/#project ?

As well as plans to support useProjectService once out of experimental phase (currently experimental as EXPERIMENTAL_useProjectService https://typescript-eslint.io/packages/parser/#experimental_useprojectservice ) which solves a few more issues (including performance) with finding the tsconfig.

@Samuel-Therrien-Beslogic Samuel-Therrien-Beslogic changed the title Support for project: true or useprojectservice: true ? Support for project: true or useProjectService: true ? May 29, 2024
@SukkaW
Copy link
Collaborator

SukkaW commented Jul 12, 2024

Is there any guide/docs about how to use/implement projectService in a third-party library (eslint-import-resolver-typescript in this case)?

@Samuel-Therrien-Beslogic
Copy link
Author

Samuel-Therrien-Beslogic commented Jul 12, 2024

Sorry I'm not that knowledgeable about how that would work, I suppose the best place to ask would be https://github.com/typescript-eslint/typescript-eslint/discussions , maybe this discussion typescript-eslint/typescript-eslint#8030 ?

Edit: I've forwarded the question: typescript-eslint/typescript-eslint#8030 (comment)

@Samuel-Therrien-Beslogic
Copy link
Author

Samuel-Therrien-Beslogic commented Sep 19, 2024

@SukkaW here's an answer for you. Hopefully it answers the question: typescript-eslint/typescript-eslint#8030 (reply in thread)

@SukkaW
Copy link
Collaborator

SukkaW commented Sep 19, 2024

typescript-eslint/typescript-eslint#8030 (reply in thread)

un-ts/eslint-plugin-import-x#40 (comment)
un-ts/eslint-plugin-import-x#40 (comment)

Due to the previous performance degradation (up to 50% slower) with parsing, I doubt the non-isolated parsing would do any good with module resolution. For now, I'd prefer not to do module resolution with type information unless the typescript-eslint team has other details that I don't know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants