Skip to content

fix: use valid config type for supporting both ESLint v8 and v9 #976

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

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Dec 6, 2024

Checks

Changes

  • switches the types back to using the Linter.FlatConfig type to ensure compatibility with both ESLint v8 and v9

Context

#973 incorrectly changed this probably because Linter.FlatConfig is marked as deprecated in v9, but it's the correct type to be using when supporting both v8 and v9 because @types/eslint (for v8) Linter.Config maps to legacy config - so right now the actual types for flat/* will change depending on the version of ESLint you've got installed.

Signed-off-by: Gareth Jones <Jones258@Gmail.com>
Copy link

codecov bot commented Dec 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.27%. Comparing base (26d360e) to head (e155d9f).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #976   +/-   ##
=======================================
  Coverage   96.27%   96.27%           
=======================================
  Files          46       46           
  Lines        2472     2472           
  Branches     1025     1030    +5     
=======================================
  Hits         2380     2380           
  Misses         92       92           

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

@G-Rath G-Rath mentioned this pull request Dec 6, 2024
1 task
@MichaelDeBoey MichaelDeBoey requested a review from Belco90 December 6, 2024 03:08
@Belco90 Belco90 added this pull request to the merge queue Dec 6, 2024
Merged via the queue into testing-library:main with commit d8e44b2 Dec 6, 2024
31 checks passed
Copy link

github-actions bot commented Dec 6, 2024

🎉 This PR is included in version 7.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@G-Rath G-Rath deleted the patch-1 branch December 17, 2024 23:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants