nodesource / ansible-nodejs-role

Ansible Role for Node.js Binary Install
MIT License
131 stars 47 forks source link

Added support for 5.x and some refactoring. #19

Open wolfeidau opened 8 years ago

wolfeidau commented 8 years ago

Looking for feedback on this from @Yanchek99 or @tphummel.

Does this look like a suitable way to deal with 0.10/0.12/4.x/5.x and more in the future?

Cheers

Yanchek99 commented 8 years ago

Would be nice to refactor the tests to check each major version. 0.10, 0.12, 4.x, and 5.x

wolfeidau commented 8 years ago

@Yanchek99 I have had a look at testing ansible before and it is a bit of a mixed bag.

I will remove the debug statement and have a look around at some way to at least asset the intended version is installed in the playbook used for testing.

Cheers.