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

Bundler support: run eye commands within a project path #5

Merged
merged 1 commit into from
Jun 8, 2015
Merged

Bundler support: run eye commands within a project path #5

merged 1 commit into from
Jun 8, 2015

Conversation

alexeymorozov
Copy link
Contributor

Hi Alex,

Thank you for the work on this gem!

I've added support for capistrano-bundler in this PR and would like to merge it to the upstream.

When using with capistrano-bundler, the working directory should contain Gemfile. Otherwise, it would fail looking for Gemfile.

I would like to answer any questions regarding this PR. Looking forward to your review.

When using with capistrano-bundler, the working directory should contain
Gemfile.
@alexsergeyev
Copy link
Owner

I use eye to manage multiple projects deployed in different locations. So, eye itself not bundled with project. If there are any way to support both cases bundled and not bundled eye I will merge this patch.

@alexeymorozov
Copy link
Contributor Author

This PR only adds the requirement to run the eye command inside the project path. It doesn't add bundler's calls anywhere in the code, because this work is done by the capistrano-bundler gem.

So it will work perfectly even with unbundled eye.

alexsergeyev added a commit that referenced this pull request Jun 8, 2015
Bundler support: run eye commands within a project path
@alexsergeyev alexsergeyev merged commit 053230d into alexsergeyev:master Jun 8, 2015
@alexsergeyev
Copy link
Owner

Yes. Everything looks fine. Thanks for PR

@alexeymorozov
Copy link
Contributor Author

Alex, thank you very much!

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

Successfully merging this pull request may close these issues.

2 participants