-
Notifications
You must be signed in to change notification settings - Fork 2
chore: use @eggjs/oxlint-config extends .oxlintrc.json #12
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: fengmk2 <[email protected]>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Co-authored-by: fengmk2 <[email protected]>
package-lock.json
Outdated
| @@ -0,0 +1,5350 @@ | |||
| { | |||
| "name": "graceful-process", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot add package-lock.json to gitignore and not commit it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added package-lock.json to .gitignore and removed it from git tracking. The file still exists locally for development but is no longer committed to the repository, following Node.js library best practices. [aa891f0]
commit: |
commit: |
Co-authored-by: fengmk2 <[email protected]>
Co-authored-by: fengmk2 <[email protected]>
This PR simplifies the oxlint configuration by extending the shared
@eggjs/oxlint-configinstead of maintaining a comprehensive manual configuration, updates the project to follow Node.js library best practices by not trackingpackage-lock.json, and upgrades to the latest oxlint v1 for improved performance and features.Changes Made
Oxlint Configuration:
@eggjs/oxlint-configas a dev dependency^0.16.10to^1.11.1for latest v1 features and improvements.oxlintrc.jsonfrom ~140 lines to 12 lines by extending the shared configenv.node: trueandenv.mocha: truefor Node.js and test environment globalsignorePatternsfor project-specific files (index.d.ts,test/fixtures/**,__snapshots__)no-empty-function: "allow"to maintain current project behaviorPackage Lock Management:
package-lock.jsonto.gitignorepackage-lock.jsonis no longer committed to the repository, following Node.js library best practicesBenefits
@eggjs/oxlint-configValidation
package-lock.jsonis properly ignored by gitThe configuration now extends from
@eggjs/oxlint-configwhile preserving the minimal project-specific customizations needed for this repository.💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.