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

@typescript-eslint requires typescript #612

Open
kunev opened this issue Feb 12, 2025 · 0 comments · May be fixed by #616
Open

@typescript-eslint requires typescript #612

kunev opened this issue Feb 12, 2025 · 0 comments · May be fixed by #616

Comments

@kunev
Copy link

kunev commented Feb 12, 2025

020bf50 adds @typescript-eslint/eslint-plugin and @typescript-eslint/parser as dependencies. However they themselves require typescript. When running yarn in a project with latest eslint-plugin-github I get the following:

 yarn
➤ YN0000: ┌ Resolution step
➤ YN0002: │ eslint-plugin-github@npm:5.1.8 [02650] doesn't provide typescript (p71046), requested by @typescript-eslint/eslint-plugin
➤ YN0002: │ eslint-plugin-github@npm:5.1.8 [02650] doesn't provide typescript (pa3e1f), requested by @typescript-eslint/parser
➤ YN0002: │ eslint-plugin-github@npm:5.1.8 [02650] doesn't provide typescript (p8f4c7), requested by typescript-eslint

and running explain for p71046:

 yarn explain peer-requirements p71046
➤ YN0000: eslint-plugin-github@npm:5.1.8 [02650] doesn't provide typescript, breaking the following requirements:

➤ YN0000: @typescript-eslint/eslint-plugin@npm:8.24.0 [bc338]     → >=4.8.4 <5.8.0 ✘
➤ YN0000: @typescript-eslint/type-utils@npm:8.24.0 [1045e]        → >=4.8.4 <5.8.0 ✘
➤ YN0000: @typescript-eslint/typescript-estree@npm:8.24.0 [8c974] → >=4.8.4 <5.8.0 ✘
➤ YN0000: @typescript-eslint/utils@npm:8.24.0 [1045e]             → >=4.8.4 <5.8.0 ✘
➤ YN0000: ts-api-utils@npm:2.0.1 [1045e]                          → >=4.8.4        ✘

Technically everything still work as the project I'm working on itself has typescript as a dependency, but I think this warning is still relevant as it means the requirements defined in package.json here are not consistent.

@gracepark gracepark linked a pull request Feb 24, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant