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

github: Publish only on release #29

Merged
merged 1 commit into from
Jan 26, 2021

Conversation

rzr
Copy link
Collaborator

@rzr rzr commented Mar 13, 2020

Change-Id: I1cc93f43a56fe9e8a7b3c37f3859a324a06f47bc
Signed-off-by: Philippe Coval rzr@users.sf.net

Change-Id: I1cc93f43a56fe9e8a7b3c37f3859a324a06f47bc
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@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

Relate-to: #28 (comment)

@rzr
Copy link
Collaborator Author

rzr commented Apr 16, 2020

Please confirm secrets are setup correctly

@rzr
Copy link
Collaborator Author

rzr commented Jun 5, 2020

Meanwhile I'll keep using:

https://www.npmjs.com/package/@abandonware/bh1750

@rzr
Copy link
Collaborator Author

rzr commented Sep 22, 2020

ping @miroRucka Please confirm secrets are setup correctly before merging this change

@miroRucka miroRucka merged commit 5cd1c25 into miroRucka:master Jan 26, 2021
@rzr
Copy link
Collaborator Author

rzr commented Jan 26, 2021

Are secrets setup as suggested at:
#20

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