search

capistrano / bundler

Bundler support for Capistrano 3.x
MIT License
219 stars 83 forks source link

Use setting to determine if test should be ran for full installation #107

Closed chadwilken closed 5 years ago

chadwilken commented 6 years ago

There was some discussion about bypassing the bundle install test command during installation in #95. I added an optional setting bundle_check_before_install that defaults to true. If you manually set to false it will perform a full installation with every deploy.

capistrano-bot commented 6 years ago
1 Error
:no_entry_sign: Please put the * Your contribution here! line back into CHANGELOG.md.

Thanks for the PR! This project lacks automated tests, which makes reviewing and approving PRs somewhat difficult. Please make sure that your contribution has not broken backwards compatibility or introduced any risky changes.

Generated by :no_entry_sign: Danger

chadwilken commented 5 years ago

I resolved the merge conflicts, let me know if there is something else you need added to this.

chadwilken commented 5 years ago

@will-in-wi I added documentation to the README and updated the logic. You're correct about the logic for the if/unless statement. I was incorrectly handling the situation when they want to run bundle check to decide if bundle install is necessary.

will-in-wi commented 5 years ago

Thanks!