Skip to content

[New] support eslint 8.x #3059

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 1 commit into from
Nov 10, 2021
Merged

[New] support eslint 8.x #3059

merged 1 commit into from
Nov 10, 2021

Conversation

Co-authored-by: Michaël De Boey <[email protected]>
Co-authored-by: Jordan Harband <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Aug 30, 2021

Codecov Report

Merging #3059 (12f33f6) into master (0f7e2e7) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 12f33f6 differs from pull request most recent head d0d5d90. Consider uploading reports for the commit d0d5d90 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3059   +/-   ##
=======================================
  Coverage   97.52%   97.52%           
=======================================
  Files         118      118           
  Lines        7914     7914           
  Branches     2833     2833           
=======================================
  Hits         7718     7718           
  Misses        196      196           
Impacted Files Coverage Δ
lib/rules/no-typos.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f7e2e7...d0d5d90. Read the comment docs.

@ljharb
Copy link
Member

ljharb commented Oct 18, 2021

Seems like babel-eslint tests can't be run on eslint 8+.

I'll land some test refactors in master that makes conditional parsers easier to do.

@ljharb
Copy link
Member

ljharb commented Oct 25, 2021

I've rebased this; the ~600 test failures are on node 12 (because it's using typescript-eslint/parser v4 latest instead of v4.0), and node 14+ (because it's using v5).

I think we need to update the TS eslint parser in a separate PR, and get those test failures resolved, and then we can rebase this again.

@ljharb
Copy link
Member

ljharb commented Oct 30, 2021

Locally I've got 3 test failures left - 2 are likely a bug in babel's eslint parser, the third is eslint/eslint#15234.

@ljharb ljharb changed the title feat: support ESLint 8.x [New] support eslint 8.x Oct 30, 2021
@aladdin-add
Copy link
Contributor

seems all the tests passed. 🎉

@ljharb
Copy link
Member

ljharb commented Nov 1, 2021

@aladdin-add yes, but we're still waiting for eslint-config-airbnb-base to be published, which is blocked on sarbbottam/eslint-find-rules#337.

@MichaelDeBoey MichaelDeBoey marked this pull request as ready for review November 9, 2021 17:14
@ljharb ljharb merged commit 9e12d2b into jsx-eslint:master Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Support ESLint 8.x
5 participants