search

cmanzana / node-publish

MIT License
13 stars 7 forks source link

Added publish in the test cases for CI #3

Closed axemclion closed 9 years ago

axemclion commented 11 years ago

Though this plugin is node-publish, it does not try publishing during the CI. Added a publish test case would help. On test complete, an unpublish script can run.

cmanzana commented 11 years ago

Yes, such a test case is needed I am trying to finalize first node-figaro (see https://github.com/cmanzana/node-figaro) in order to be able to add node credentials into CI without the need of having them in the repository I have not been able to work on either of these repos for quite a while now

axemclion commented 11 years ago

How about using Travis Secure environment variables? I use that for my repo here - https://github.com/axemclion/grunt-saucelabs/blob/master/Gruntfile.js#L94

https://github.com/axemclion/grunt-saucelabs/blob/master/.travis.yml

cmanzana commented 11 years ago

that is actually the point of node-figaro: to make the usage of secure environment variables straight forward but as said, I have not been able to really test it (I did the node-figaro code before travis supported secure environment variables, and since then I have not had the time to check it)