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

Docs: Out-of-date TS error codes in eslint-recommended.ts #7582

Closed
2 tasks done
JoshuaKGoldberg opened this issue Aug 31, 2023 · 2 comments · Fixed by #7590
Closed
2 tasks done

Docs: Out-of-date TS error codes in eslint-recommended.ts #7582

JoshuaKGoldberg opened this issue Aug 31, 2023 · 2 comments · Fixed by #7590
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating good first issue Good for newcomers

Comments

@JoshuaKGoldberg
Copy link
Member

Before You File a Documentation Request Please Confirm You Have Done The Following...

Suggested Changes

https://github.com/typescript-eslint/typescript-eslint/blob/939e42f05afa9536e03923b8829f060cdf35ada0/packages/eslint-plugin/src/configs/eslint-recommended.ts includes comments referring to corresponding TS error codes for rules. Example:
https://github.com/typescript-eslint/typescript-eslint/blob/939e42f05afa9536e03923b8829f060cdf35ada0/packages/eslint-plugin/src/configs/eslint-recommended.ts#L11C17-L11C17

Some of those error codes are out of date: eslint/eslint#17500 (comment). Someone should go through and update the TS codes to whatever they are now as of the latest stable version of Typescript.

Affected URL(s)

https://github.com/typescript-eslint/typescript-eslint/blob/939e42f05afa9536e03923b8829f060cdf35ada0/packages/eslint-plugin/src/configs/eslint-recommended.ts

(Technically comments rather than website, but they're used by some as docs too)

@JoshuaKGoldberg JoshuaKGoldberg added good first issue Good for newcomers documentation Documentation ("docs") that needs adding/updating accepting prs Go ahead, send a pull request that resolves this issue labels Aug 31, 2023
@girishsontakke
Copy link
Contributor

@JoshuaKGoldberg I would like to work on this fix, please assign this issue to me

@JoshuaKGoldberg
Copy link
Member Author

https://typescript-eslint.io/contributing/issues#issue-claiming - but you're welcome to send a PR. Which I see you did 😄 thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating good first issue Good for newcomers
Projects
None yet
2 participants