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

Setup CD / CI #28

Merged
merged 4 commits into from
Mar 13, 2020
Merged

Conversation

rzr
Copy link
Collaborator

@rzr rzr commented Mar 5, 2020

You will need to add a secret with your NPM token like:

NPM_AUTH_TOKEN=B4DC0D3-B4DC-0D3B-4DC0-BADC0DE..

Then one commit a version using "Release X.Y.Z" it will be published automatically

#20

rzr added 4 commits March 5, 2020 14:44
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Origin: https://raw.githubusercontent.com/actions/starter-workflows/master/ci/node.js.yml
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Change-Id: I3a7004302dace975cb6938efca0d6f19ae7db810
Origin: https://github.com/marketplace/actions/publish-to-npm
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Change-Id: I25337e6b6272ad4fd9c3a1bcca486a921192b9ff
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Change-Id: Ib20cdbeb982f0fd3a09eaf5929b9006de8bd9ee1
@rzr
Copy link
Collaborator Author

rzr commented Mar 5, 2020

I confirm this change set is working

https://www.npmjs.com/package/@abandonware/bh1750/v/0.0.7-3

was released using this single change:

abandonware@01477ba

So it's safe to merge, please confirm key is enabled and I will push a change for v0.0.7

Then I will abandon the abandonware fork.

@miroRucka miroRucka merged commit 350077f into miroRucka:master Mar 13, 2020
@rzr
Copy link
Collaborator Author

rzr commented Mar 13, 2020

Please confirm you generated a token from:

https://www.npmjs.com/settings/miro.rucka/#tokens

And pasted value to NPM_AUTH_TOKEN key on github page:

https://github.com/miroRucka/bh1750/settings/secrets

Then I will push a Release change

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.

2 participants