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

Update travis api_key for npm deploys #1667

Merged
merged 2 commits into from Jan 16, 2019
Merged

Update travis api_key for npm deploys #1667

merged 2 commits into from Jan 16, 2019

Conversation

mjmahone
Copy link
Contributor

This api key was out of date preventing us from doing automatic version deployments.

Created the api key by running "travis encrypt": https://docs.travis-ci.com/user/deployment/npm/#npm-auth-token

.travis.yml Outdated
branch: master
- provider: npm
skip_cleanup: true
email: lee@leebyron.com
Copy link
Member

Choose a reason for hiding this comment

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

@mjmahone I think you need to write email that you registered in NPM:
image

Copy link
Contributor Author

@mjmahone mjmahone Jan 16, 2019

Choose a reason for hiding this comment

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

Hmmm I'm not sure it's required, as we both have publish access, but that would make sense. Also probably better for the person whose auth token is being used to get any emails associated with it.

.travis.yml Outdated
skip_cleanup: true
email: lee@leebyron.com
api_key:
secure: VrDxiDK9vhe0F8WulJ+HcAkMwuvqIhtXYethlJBwkS9N57F4RuxzCJE0VuDDS90qMAewA450lfT5caCazir4opvpZebB16iy8/9J2ymhBj6bJs9rm/B3Xg/xQSLOWAmdq1H4lPF6CihQ5LR57BZbi9Rx3f5JOeXgiqQNASJH+mrXiQh3qZ9Ry8YJXhvcUDQNX57ooSBeGYXf5PcKsEUF0Qs6XLUCUdOztMrtA+sIZkQxAaEBBG7tkE6LdF3OJl2ORNMig05juyfeevcGk6g4XnoIA4gvnW1KHToLvuGWdoCHAW1N269JAaH4bL2JfcA5A7KXbZVwif0jIJz47dmOsnIMlMdSqBpw7zSRSq2Ikv1Vb5XRPeVk0CK0d2kUZej6Jb9+/Jed39chfyAAMVM4Wba8UcTKbOIqv7Kl1SmmgBOGveK7k4fpffbWdofi9avuqzeftdFrM1FULsX5Wne9n3xuJDvVLW0YRB2+FVmfwGNwzW5X/bzgLDRWB5W5JL60FXt91Hrk5bIi0GcPx6n0Ls7IJ9U6r5kWfcuTYl0gxOiWQDJ0OCXBXA5taIO6/AX/d9af4VhGc/EVxuIpUaW52nZOL49KG/3tYx39mDCqRf7w7/658RVXNO/yLy6b/QPY7+3Lua9y88xLxru3bI9gpNUdnYUFIehktuiAkuunWoA=
Copy link
Member

Choose a reason for hiding this comment

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

@mjmahone Can you please copy this token into old .travis.yml?
All other changes in the file are results of travis CLI and shouldn't influence publishing.
It will keep GitHub history clean.

@mjmahone mjmahone merged commit 51c67dc into master Jan 16, 2019
@IvanGoncharov IvanGoncharov deleted the update-travis branch January 16, 2019 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants