Skip to content

feat: support eslint ^9 #3726

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

li-jia-nan
Copy link

feat: support eslint ^9

Copy link

codecov bot commented Apr 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.76%. Comparing base (e4ecbcf) to head (4b94a6b).
Report is 28 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3726   +/-   ##
=======================================
  Coverage   97.76%   97.76%           
=======================================
  Files         133      133           
  Lines        9467     9467           
  Branches     3467     3467           
=======================================
  Hits         9255     9255           
  Misses        212      212           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Standard8
Copy link

I'm not a maintainer here, but this doesn't seem to be enough - see the comments on issue #3699

@li-jia-nan
Copy link
Author

Sorry, I didn't notice that there was already an issue here, I closed this PR.

@li-jia-nan li-jia-nan closed this Apr 6, 2024
@Standard8
Copy link

Oh, sorry, I don't think you have to close it, but it does seem there's a bit more work than just bumping the compatible versions.

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/eslint@9.5.0 environment Transitive: eval, filesystem, shell, unsafe +80 9.5 MB eslintbot

View full report↗︎

@ljharb ljharb force-pushed the master branch 2 times, most recently from 380e32c to 51d342b Compare July 4, 2024 15:25
@ljharb
Copy link
Member

ljharb commented Jul 16, 2024

I believe this was obsoleted by #3759.

@ljharb ljharb closed this Jul 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants