-
Notifications
You must be signed in to change notification settings - Fork 60
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
feat: update package.json linter config #694
Conversation
hugomrdias
commented
Dec 15, 2020
- add a few new rules
- update engine to =>12
- add double license option
- add a few new rules - update engine to =>12 - add double license option
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.
LGTM
"require-version": "error", | ||
"bin-type": "error", | ||
"files-type": "error", | ||
"version-type": "error", | ||
"valid-values-engines": ["error", [ | ||
{ | ||
"node": ">=10.0.0", | ||
"node": ">=12.0.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.
We only support LTS and Current so this should be >=14.0.0
@@ -5,14 +5,15 @@ | |||
"require-bugs": "error", | |||
"require-bundledDependencies": "off", | |||
"require-config": "off", | |||
"require-contributors": "off", | |||
"require-contributors": "error", |
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.
This will error when the contributors field is absent, right? ipfs has contributors in the root package.json but not always in the modules under /packages
as otherwise we'd have to compare the paths of edits during our update-contributors
step.