-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
chore: v6.0.0 #5886
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
chore: v6.0.0 #5886
Conversation
Thanks for the PR, @JoshuaKGoldberg! typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community. The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately. Thanks again! 🙏 Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitors per day. |
✅ Deploy Preview for typescript-eslint ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Is it time to drop ESLint v6 support from peer deps? It was released in 2019: https://eslint.org/blog/2019/06/eslint-v6.0.0-released/
|
What do you think of strictly-defining the Node API with This would follow the lead of ESLint core and other ESLint plugins so that users cannot reach in and import/depend on arbitrary files inside the package.
https://nodejs.org/api/packages.html#main-entry-point-export |
We're all for it - #5586 |
Sounds good to me! Last major we dropped v5 support - so happy to do it. |
In case you guys aren't already tracking it, I see a lot of "TODO" / |
I also see 5 deprecated rules that could be deleted. |
@bmish thanks for pointing these out! Anything that we've marked as deprecated (should be removed in general) deserves an issue tracking it. If you see anything that doesn't yet have one, feel free to file it! I want to eventually do a sweep but haven't had the time. 🥲 |
) * refactor(utils)!: removed `TRuleListener` generic from the `createRule` * refactor!: removed `TRuleListener` generic from the `CLIEngine` and `RuleCreateFunction` * chore: document and refactor 'extra' to 'parserSettings' (#5834) * chore(website): fix renamed Sponsorship docs link (#5882) * docs: Mention wide globs performance implications in monorepos docs and parser README (#5864) * docs: Mention wide globs performance implications in monorepos docs and parser readme * Update docs/linting/typed-linting/MONOREPOS.md Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com> Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com> Co-authored-by: Adnan Hashmi <56730784+adnanhashmi09@users.noreply.github.com>
* feat: update TSImportType node * fix: update visitor keys * chore: document and refactor 'extra' to 'parserSettings' (#5834) * chore(website): fix renamed Sponsorship docs link (#5882) * docs: Mention wide globs performance implications in monorepos docs and parser README (#5864) * docs: Mention wide globs performance implications in monorepos docs and parser readme * Update docs/linting/typed-linting/MONOREPOS.md Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com> Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com> Co-authored-by: Adnan Hashmi <56730784+adnanhashmi09@users.noreply.github.com>
* feat(scope-manager): ignore ECMA version * chore: document and refactor 'extra' to 'parserSettings' (#5834) * chore(website): fix renamed Sponsorship docs link (#5882) * Remove much more * Fix WebLinter lint * docs: Mention wide globs performance implications in monorepos docs and parser README (#5864) * docs: Mention wide globs performance implications in monorepos docs and parser readme * Update docs/linting/typed-linting/MONOREPOS.md Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com> * chore: add auto-canary release for v6 (#5883) Co-authored-by: Adnan Hashmi <56730784+adnanhashmi09@users.noreply.github.com>
…TSInterfaceDeclaration and TSModuleDeclaration (#4863) * chore: remove invalid properties from ast nodes * chore: remove invalid code in scope-manager and typescript-estree * chore: re-write snapshots that were using invalid properties * feat: remove modifiers union from ast types Co-authored-by: Juan García <juank1809@gmail.com> Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com>
* feat(eslint-plugin): apply final v6 changes to configs * Ah, fixed new lint rule violations internally * no-mixed-enums strict * Apply suggestions from code review Co-authored-by: Brad Zacher <brad.zacher@gmail.com> * Fix up build & lint issues * Never mind, prefer nullish coalescing --------- Co-authored-by: Brad Zacher <brad.zacher@gmail.com>
* chore: enabled stylistic-type-checked internally * Reduced scope of changes * Updated snapshots * Update packages/typescript-estree/tests/lib/convert.test.ts Co-authored-by: rubiesonthesky <2591240+rubiesonthesky@users.noreply.github.com> * More deeplyCopy removals * Updated test snapshots --------- Co-authored-by: rubiesonthesky <2591240+rubiesonthesky@users.noreply.github.com>
* feat(eslint-plugin): final final config changes for v6 * yarn generate:configs * Fix unused lint suppression complaint and test snapshot * unknown, not any, in recommended-does-not-require-program * Updated endColumn
|
…irst operands are unrelated to the rest of the chain and add type info (#6397)
…ver match error types (#6869) * fix(type-utils): treat intrinsic types as if they are from lib and never match error types * Update packages/type-utils/tests/TypeOrValueSpecifier.test.ts * chore: bump ts-api-utils to 1.0.1 * refactor: use isIntrinsicErrorType --------- Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>
BREAKING CHANGE: v6, hooray! 🥳
This PR contains all of the changes for v6.0.0.
We will
rebasemerge this ontomain