search

Turistforeningen / wercker-npm-publish

Publish NPM packages through Wrecker CI
https://app.wercker.com/#applications/557991deeb62408b7f002d43/tab/details
MIT License
4 stars 2 forks source link

Use environment var exported by Wercker #1

Closed mariosant closed 7 years ago

mariosant commented 7 years ago

Private publishing is currently broken, because NPM_CONFIG_ACCESS is never exported. This PR switches to the exported WERCKER_NPM_CONFIG_ACCESS.

Starefossen commented 7 years ago

Thanks! Could you increment the version in wercker-step.yml as well?

mariosant commented 7 years ago

Sure thing, doing this right now.

mariosant commented 7 years ago

@Starefossen would it be worth to update package.json version as well, to match the wercker-step.yml?

Starefossen commented 7 years ago

Thanks, the package.json is just a test file used by test.sh. The version there just arbitrary.

Starefossen commented 7 years ago

Thanks for your contribution @mariosant 🎉 I have added you to the collaborator team for this project.

mariosant commented 7 years ago

Thanks for adding me @Starefossen ! I am an actual user of this one and I see huge value within this step. I am more than happy to contribute on a standard basis for this one.