Skip to content

[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

Merged
merged 66 commits into from
Jun 26, 2025

Conversation

D4N14L
Copy link
Member

@D4N14L D4N14L commented May 3, 2025

Summary

  • Adds support for ESLint 9 to the @rushstack/heft-lint-plugin.
  • Add new flat configs into @rushstack/eslint-config
  • Creates new eslint-9-test project to test the plugin against ESLint 9
  • [TODO] Updates projects that use the repo version of the ESLint plugin and configs to consume the new flat configs
  • [TODO] Adds ESLint 9 support to eslint-bulk-suppressions

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.

@benjamind
Copy link
Contributor

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 :)

@D4N14L D4N14L marked this pull request as ready for review June 11, 2025 18:02
@D4N14L
Copy link
Member Author

D4N14L commented Jun 11, 2025

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

@iclanton iclanton merged commit 549b5e2 into microsoft:main Jun 26, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Closed in Bug Triage Jun 26, 2025
@iclanton iclanton deleted the user/danade/UpdateEslint2 branch June 26, 2025 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

4 participants