Skip to content
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

Automates NPM publication #48

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jfmercer
Copy link
Contributor

Automates NPM publication

Description

  • automates NPM publication
  • updates CHANGELOG for new release
  • fixes CHANGELOG formatting
  • sets new release: 0.6.1

Test Script

  • None. CI must pass.
  • Note: the new version will not be published until I update the tag on Github, which I'll do as soon as this PR is merged.

Additionally,

* updates CHANGELOG for new release
* fixes CHANGELOG formatting
* sets new release: 0.6.1
@@ -14,6 +14,15 @@ before_script:
script:
- gulp

deploy:
provider: npm
email: "john@radify.io"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just set up shipping@radify.io as an email group that we can use for stuff like this, so that no one becomes a single point of failure.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nateabele If we use shipping@radify.io, that email will also need to have a Github account, an NPM account, and an NPM API key. Regarding, npm publish, no one person can become a single point of failure so long as multiple contributors are listed in package.json and on the npmjs.com website. Regarding automated Travis publication, my credentials could become a point of failure if I changed by NPM API key, but even that could be fixed with a simple change of the .travis.yml. Should we go ahead & switch to shipping@radify.io? Additionally, I'm looking into NPM Organizations, which seems to be a new thing:tm:, and might solve problems like this for us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants