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
RFC: Support eslint@9.0.0 #735
Draft
TylerJang27
wants to merge
7
commits into
main
Choose a base branch
from
tyler/eslint-9
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
⏱️ 1h 59m total CI duration on this PR
|
Ultimately, there are 2 existing issues warranting this change (or a similar one):
|
This was referenced Apr 15, 2024
TylerJang27
added a commit
that referenced
this pull request
Apr 15, 2024
Fixes [failures](https://github.com/trunk-io/trunk/actions/runs/8659672695/job/23746050111) and cleans up dependabot ignores for `eslint` until #735 is resolved. [Successful run](https://github.com/trunk-io/trunk/actions/runs/8693488278/job/23840365226)
Merged
TylerJang27
added a commit
that referenced
this pull request
Apr 23, 2024
Offers some temporary fixes to make `main` green and provide a migration guide until we have a long-term story for #735. Also includes a node fix with [executing `.bat` files](https://github.com/trunk-io/plugins/actions/runs/8702103641/job/23865448625) related to prebuild/prebuildify#83
TylerJang27
force-pushed
the
tyler/eslint-9
branch
2 times, most recently
from
May 8, 2024 18:44
2fa6e8a
to
aed6721
Compare
TylerJang27
force-pushed
the
tyler/eslint-9
branch
from
May 9, 2024 09:07
aed6721
to
948302b
Compare
❌ 427 failed 💊 0 quarantined ✅ 3 passed 🕐 436 new ⋅ (learn more) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Eslint@9.0.0 was released and with it a major change to how eslint is configured. The migration process is nontrivial and may not be an option/priority for every project. Additionally, upgrading will loudly fail unless you have accounted for the config. Accordingly, I have done the following:
direct_configs
a property ofcommands
, allowing us to version the configs that are applicable for lintereslint
tests to handle pre- and post- eslint9eslint9
flat configeslint9
This will require a version bump and minimum requirement increase to support.