-
Notifications
You must be signed in to change notification settings - Fork 20
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
👌 IMPROVE: [BREAKING] use eslint-plugin-jsdoc@39 to support Node.js 18 #64
Conversation
@@ -18,7 +18,7 @@ | |||
"@typescript-eslint/parser": "^5.4.0", | |||
"eslint-plugin-eggache": "^1.0.0", | |||
"eslint-plugin-import": "^2.25.3", | |||
"eslint-plugin-jsdoc": "^37.0.3", | |||
"eslint-plugin-jsdoc": "^39.3.0", |
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.
Only support Node.js >= 14 Remove strict rule
@atian25 直接发大版本,只支持 node >= 14 |
去掉了 |
This pull request fixes 1 alert when merging 9aa6838 into c498d30 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging a8b7c6e into c498d30 - view on LGTM.com fixed alerts:
|
Codecov Report
@@ Coverage Diff @@
## master #64 +/- ##
=============================================
+ Coverage 0 100.00% +100.00%
=============================================
Files 0 29 +29
Lines 0 3285 +3285
Branches 0 94 +94
=============================================
+ Hits 0 3285 +3285
Continue to review full report at Codecov.
|
This pull request fixes 2 alerts when merging aad8fb0 into c498d30 - view on LGTM.com fixed alerts:
|
12.0.0 |
Only support Node.js >= 14
Remove strict rule