-
Notifications
You must be signed in to change notification settings - Fork 18
Conversation
@@ -9,7 +9,8 @@ | |||
"release": "lerna publish", | |||
"release:force": "lerna publish --force-publish", | |||
"test": "lerna exec --scope='*' -- npm run test", | |||
"lint": "lerna exec --scope='*' -- npm run lint" | |||
"lint": "lerna exec --scope='*' -- npm run lint", | |||
"cover": "exit 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.
This will be implemented later
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.
Before the next release we will need to restore the browser bundle formerly created in the NPM package ("prepublish": "npm run ci && npm run build && npm run web",
). Perhaps we can create separate issue to track.
@@ -21,6 +22,6 @@ | |||
"mocha": "^5.0.2" | |||
}, | |||
"engines": { | |||
"node": ">=4" | |||
"node": ">=6" |
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.
The CHANGELOG.md
file from the repo seems to be missing, We should mark a breaking change that Node 4 support was removed.
d34f104
to
91390b2
Compare
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 just want to see the following change and we're good to merge this. Just so we don't forget about this breaking change before our next release.
- Add CHANGELOG entry for Master that there is a Breaking change because Node 4 is now unsupported.
91390b2
to
a276b7a
Compare
Fix CHANGELOG format consistency
I used the default github's .gitignore for Node.js