jsx-eslint / eslint-plugin-react

React-specific linting rules for ESLint
MIT License
8.86k stars 2.75k forks source link

[Feat]: provide compatibility with eslint v9 #3727

Open antongolub opened 2 months ago

antongolub commented 2 months ago

relates #3699 superseeds #3726

antongolub commented 2 months ago

Oh... One more.

ESLint: 9.0.0

TypeError: context.getScope is not a function
Occurred while linting /gh/forks/eslint-plugin-react/configs/all.js:3
Rule: "import/no-amd"
    at CallExpression (/gh/forks/eslint-plugin-react/node_modules/eslint-plugin-import/lib/rules/no-amd.js:26:25)
    at ruleErrorHandler (/gh/forks/eslint-plugin-react/node_modules/eslint/lib/linter/linter.js:1145:48)
    at /gh/forks/eslint-plugin-react/node_modules/eslint/lib/linter/safe-emitter.js:45:58
antongolub commented 2 months ago

Awaits https://github.com/import-js/eslint-plugin-import/pull/2996

socket-security[bot] commented 2 months ago

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

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

View full report↗︎

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 99.28571% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 97.57%. Comparing base (e4ecbcf) to head (d87ada8). Report is 2 commits behind head on master.

Files Patch % Lines
lib/util/ast.js 87.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3727 +/- ## ========================================== - Coverage 97.76% 97.57% -0.19% ========================================== Files 133 133 Lines 9467 9496 +29 Branches 3467 3480 +13 ========================================== + Hits 9255 9266 +11 - Misses 212 230 +18 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.