Skip to content
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

Add more options to the exec command #102

Closed
nickserv opened this issue Apr 6, 2014 · 2 comments
Closed

Add more options to the exec command #102

nickserv opened this issue Apr 6, 2014 · 2 comments

Comments

@nickserv
Copy link
Collaborator

nickserv commented Apr 6, 2014

--all

In order to keep certain functionality if we remove git as a runtime dependency, we should add a homesick exec --all flag to make it easier to run commands on every cloned castle.

As an example, here's what the --all flag for homesick pull looks like currently:

method_option :all,
              type: :boolean,
              default: false,
              required: false,
              desc: 'Update all cloned castles'

--pretend and --quiet

This will be especially useful for #99.

@nickserv nickserv changed the title Add a --all option to homesick exec Add a --all option to the exec command Apr 6, 2014
@nickserv nickserv changed the title Add a --all option to the exec command Add more options to the exec command Apr 7, 2014
@JCook21
Copy link
Collaborator

JCook21 commented Apr 12, 2014

@nicolasmccurdy I have some code that adds this functionality, just need to add some tests. I'll hopefully get a PR together during this weekend.

@nickserv
Copy link
Collaborator Author

@JCook21 fixed this in #105. His implementation uses an exec_all command instead of an --all flag, which makes more sense when dealing with castle names. However, in my opinion, specifying castle names should eventually be done with a homesick-wide flag, which means that exec_all could change into an --all option for exec instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants