-
Notifications
You must be signed in to change notification settings - Fork 1
chore: update oxlint config to use @eggjs/oxlint-config #30
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
chore: update oxlint config to use @eggjs/oxlint-config #30
Conversation
Co-authored-by: fengmk2 <[email protected]>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #30 +/- ##
=======================================
Coverage 91.40% 91.40%
=======================================
Files 15 15
Lines 1443 1443
Branches 291 291
=======================================
Hits 1319 1319
Misses 124 124 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Join our Discord community for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
[skip ci] ## <small>2.5.1 (2025-08-09)</small> * fix: patch File to support test on Node.js 18 (#31) ([93edf11](93edf11)), closes [#31](#31) * chore: remove eslint deps (#28) ([46b4acf](46b4acf)), closes [#28](#28) * chore: update oxlint config to use @eggjs/oxlint-config (#30) ([798a8d1](798a8d1)), closes [#30](#30)
|
🎉 This PR is included in version 2.5.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This PR updates the project's oxlint configuration to use the shared
@eggjs/oxlint-configpackage instead of maintaining a large inline configuration.Changes Made
@eggjs/oxlint-config@^1.0.0as a dev dependency.oxlintrc.jsonwith a minimal 15-line extends-based configurationmochaenvironment (in addition tonodefrom base config)ignorePatternsforindex.d.ts,test/fixtures/**, and__snapshots__no-consolerule to "warn" andno-empty-functionto "allow" to match original project behaviorBenefits
import/consistent-type-specifier-styleVerification
npm run prepublishOnlyoxlint --fixfunctionality works as expectedThe configuration now leverages the shared, community-maintained rules while preserving the project's specific requirements.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.