capistrano / rvm

MIT License
140 stars 47 forks source link

Allow for overriding of RVM roles #41

Closed cheald closed 10 years ago

cheald commented 10 years ago

It is sometimes useful to limit RVM checks and usage to certain roles. This change simply allows for an :rvm_roles variable which may be set to specify the roles that RVM operates on. It defaults to :all.

mpapis commented 10 years ago

can you also update README?

cheald commented 10 years ago

Yup, can do. Is the :rvm_roles key good with you?

mpapis commented 10 years ago

sounds good, but I'm not officially maintaining this repo so it's not up to me ;)

cheald commented 10 years ago

Ah, heh, of course. :)

kirs commented 10 years ago

That would be useful option :+1: Thanks!

cheald commented 10 years ago

Fixed and squashed.

kirs commented 10 years ago

Thanks!

mpapis commented 10 years ago

one question, did anyone test it without rvm in one of stages? in rvm1-capistrano3 I had to use an extra check => https://github.com/rvm/rvm1-capistrano3/compare/49b60d9a06ea2cf0e20743ec7724b106fcf82ca1...3e3fb5c375c282479f514cd8a22e0aee2e78539c?w=1 I guess should be updated before #52