-
Notifications
You must be signed in to change notification settings - Fork 82
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
Auto npm publish action and master update #178
base: master
Are you sure you want to change the base?
Conversation
Added only on tag should the action run
Update from master
npm install updates from master
This reverts commit 50b6b55.
I think there is an issue with the postinstall.js; as I've been trying to install this in a local registry for a project. I don't know much about what this is doing though.
|
Strange, that you’re not getting any error output besides the existence of an error in that file 🤔 |
8d4bd4a
to
c8e81b0
Compare
@sclausen I updated this pr with your recent changes and the postinstall.js issue I saw above doesn't seem to be a problem. It looks like travis-ci failed due to docker secrets; will probably work for you if you have those set. |
This is basically PR #161 but I added the requested feature of only working when a tag is made. This PR used the original PR's branch, but then I updated that branch from current master.
I'm not able to do a end-to-end test with this but I use this same methodology in my project.
In the .yml in this request, I added but commented it out if you wanted to restrict it to just the master branch based on this