-
-
Notifications
You must be signed in to change notification settings - Fork 243
fix: respect existing eslint.config.ts, eslint.config.cts, eslint.config.mts files #2458
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
Conversation
View your CI Pipeline Execution ↗ for commit bf3c25e.
☁️ Nx Cloud last updated this comment at |
b7d82af
to
4492b15
Compare
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (73.80%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2458 +/- ##
==========================================
+ Coverage 92.84% 92.98% +0.14%
==========================================
Files 195 195
Lines 4093 4105 +12
Branches 954 959 +5
==========================================
+ Hits 3800 3817 +17
+ Misses 226 221 -5
Partials 67 67
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
4492b15
to
3ee1dc3
Compare
NOTE: This PR does not yet handle generating such files, but they will be respected if they already exist, that's why I'm marking this as a fix.
Just like with ESLint directly, it is up to you to ensure that an appropriate version of
jiti
is installed to make it work (ESLint will error if you don't have it, as shown in the e2e test)Fixes #2205