-
Notifications
You must be signed in to change notification settings - Fork 636
[heft-lint-plugin] Add support for ESLint 9 #5219
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
Conversation
…er/danade/UpdateEslint2
rigs/local-node-rig/profiles/default/includes/eslint/flat/mixins/friendly-locals.js
Outdated
Show resolved
Hide resolved
…er/danade/UpdateEslint2
What would it take to get this over the finish line? Am setting up a new repository with eslint today and would like to use eslint 9 if I can to avoid having to look at this again in a few months time :) |
The PR is in a workable state, though it needs reviews. It's... quite a large PR though, so it's (reasonably) taking some time for people to get around to it. Once I have some reviews, I'll merge in the latest changes in the repo and update to address feedback. CC @IanC , @dmichon-msft |
…er/danade/UpdateEslint2
eslint/eslint-patch/src/eslint-bulk-suppressions/generate-patched-file.ts
Outdated
Show resolved
Hide resolved
eslint/eslint-patch/src/eslint-bulk-suppressions/generate-patched-file.ts
Show resolved
Hide resolved
eslint/eslint-patch/src/eslint-bulk-suppressions/generate-patched-file.ts
Show resolved
Hide resolved
…er/danade/UpdateEslint2
Summary
NOTE: Once merged, a publish and bump of decoupled dependencies will be required to update the remaining projects. The new configs should be moved into the decoupled rig, which currently is the source for the configs for the
local-node-rig
configs.How it was tested
This PR.
Impacted documentation
Any ESLint-related docs will need to be updated to indicate support.