-
Notifications
You must be signed in to change notification settings - Fork 37
Create skeleton for package-json-lint project #541
Conversation
## Versioning | ||
|
||
package-json-lint is considered to be stable and will follow [SemVer](http://semver.org/) for versioning. | ||
|
||
1. MAJOR versions represent breaking changes | ||
2. MINOR versions represent added functionality in a backwards-compatible manner | ||
3. PATCH versions represent backwards-compatible bug fixes | ||
|
||
Consult the component CHANGELOGs, related issues, and PRs for more information. |
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.
Can this be removed since it's describe at the root level?
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.
I was mimicking existing packages. As I think about it, it wouldn't show up in the npm page if we didn't include that (if that matters): e.g. https://www.npmjs.com/package/@cerner/terra-cli
@@ -3,6 +3,7 @@ | |||
"command": { | |||
"publish": { | |||
"ignoreChanges": [ | |||
"packages/package-json-lint/**", |
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.
Should this be package-json-linter? instead of just lint?
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.
It's eslint and stylelint not eslinter or stylelinter. I was trying to follow that pattern.
Co-authored-by: Emily Rohrbough <emilyrohrbough@users.noreply.github.com>
Summary
This is just the initial commit of the package-json-lint project. It is set up not to be released and will be built out as a part of #540
Fixes #538