Skip to content

Moved the ci:all rake task invocation out of band#24

Merged
joshpencheon merged 4 commits into
NHSDigital:masterfrom
timgentry:feature/with_clean_rbenv
Oct 17, 2018
Merged

Moved the ci:all rake task invocation out of band#24
joshpencheon merged 4 commits into
NHSDigital:masterfrom
timgentry:feature/with_clean_rbenv

Conversation

@timgentry

Copy link
Copy Markdown
Contributor

The CI server used to invoke the ci:all rake task from within the CI rake task, causing it to be brittle in the face of bundle and/or migration db schema changes.

This is a breaking change as projects no longer need to discard their own changes or bundle install.

@timgentry

Copy link
Copy Markdown
Contributor Author

This PR could be squashed and merged.

@joshpencheon joshpencheon merged commit ec3d1f3 into NHSDigital:master Oct 17, 2018
timgentry added a commit that referenced this pull request Oct 19, 2018
* # Added with_clean_rbenv and shelling out to bundle install with guard clause for projects with no Gemfile

* # Additional guard clause to speed up bundle update with no Gemfile changes

* # Added missing require

* # discarding local changes e.g. as a result of migrations or bundle install
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