-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Change Request: Add @types/eslint dependency #105
Labels
enhancement
New feature or request
Comments
Let's keep the conversation on #104. There's no need for a separate issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Which packages would you like to change?
@eslint/compat
@eslint/config-array
@eslint/core
@eslint/migrate-config
@eslint/object-schema
What problem do you want to solve?
Prevent type errors when using this plugin in TS files. Follow up from #104.
What do you think is the correct solution?
This package exports type definitions in d.ts files referencing types from the
eslint
package. Therefore, it should include@types/eslint
as public dependency.As stated in #104, this package should work with
@types/eslint
v8 and v9, therefore the dependency should be just"*"
as done in https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/eslint__js/package.json#L9Participation
Additional comments
No response
The text was updated successfully, but these errors were encountered: