-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
Close #1470: upgrade eslint #1472
Conversation
🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎ To accept the risk, merge this PR and you will not be notified again.
Next stepsWhat is a critical CVE?Contains a Critical Common Vulnerability and Exposure (CVE). Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies. Take a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with
|
commit: |
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.
Flat config is fine!
LGTM.
@samchon maybe we can add lint/format check in github actions |
OK, about the github actions eslint filtering, I'll wait your PR. Thanks for reviewing. |
@samchon I found that eslint config is wrong, I'm sorry. |
eslint
has been changed a lot, so hard to follow it up.Can you review this one? @ryoppippi
This pull request includes several updates to dependencies and configuration files to improve the project's setup and ensure compatibility with the latest tools and libraries. The most important changes include updating dependencies in
package.json
, renaming and modifying the ESLint configuration file, and adjusting ESLint scripts.Dependency Updates:
@samchon/openapi
dependency version range to ">=2.4.0 <3.0.0" inpackage.json
.@typescript-eslint/eslint-plugin
and@typescript-eslint/parser
to version "^8.1.0" inpackage.json
.eslint-plugin-deprecation
version "^3.0.0" topackage.json
.Configuration Changes:
.eslintrc.cjs
toeslint.config.cjs
and addeddeprecation
plugin to the ESLint configuration.eslint.config.cjs
to disable certain rules and reorder file patterns.Script Adjustments:
package.json
to remove file patterns from the command.