Conversation
2383756
to
3066d7e
Compare
package.json
Outdated
}, | ||
"devDependencies": { | ||
"eslint": "^5.3.0", | ||
"semantic-release": "^15.9.8" |
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'd remove this - we're using npx
above anyway
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.
How will the module be available if it isn't installed? Or maybe install it as part of the travis.yml
?
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.
How will the module be available if it isn't installed?
That's sorta the point of npx
- running binaries from modules that are not necessarily installed into the project. https://www.npmjs.com/package/npx#description :)
.travis.yml
Outdated
node_js: | ||
- "8" | ||
after_success: | ||
npm run semantic-release |
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.
lol, noooo.. you've gone the other way!
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.
Let's use npx
here, and drop it from our deps
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.
🤣
6047aa0
to
49d99b4
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.
Looks good, but I think the lockfile it out-of-date
49d99b4
to
d93a4b5
Compare
🎉 This PR is included in version 1.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.