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

Allow create release build stage to fail without failing the build #524

Merged
merged 4 commits into from
Jul 13, 2017

Conversation

gemmaleigh
Copy link
Contributor

@gemmaleigh gemmaleigh commented Jul 13, 2017

Add a script to check if the VERSION.txt file has been updated in packages/govuk-elements-sass/, if it hasn't been updated - exit without creating a release.

Allow failures for the "create release" build stage.
This should be allowed to fail - without marking the entire build as a
failure - as we’re only deploying when the packages directory has been
updated.

This can be seen working here -
where the build stage "create release" fails (and that's fine, that's what we expect unless we've updated the packages directory and updated VERSION.txt to release a new version) but the overall build succeeds.

@gemmaleigh gemmaleigh temporarily deployed to govuk-elements-review-pr-524 July 13, 2017 11:15 Inactive
@gemmaleigh gemmaleigh changed the title [WIP] Allow create release build stage to fail without failing the build Allow create release build stage to fail without failing the build Jul 13, 2017
Ensure Travis aborts without creating a release if version hasn't
changed.

If it has changed, proceed to create release. Otherwise exit.
This should be allowed to fail, without marking the entire build as a
failure - as we’re only creating a release on tagged commits - once the
version.txt file in packages/govuk-elements-sass/ has been updated. The
check-release-version script will exit if the version hasn’t been
changed.
This ensures all build scripts are moved into the scripts directory.
Copy link
Contributor

@36degrees 36degrees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@gemmaleigh gemmaleigh merged commit dd398bf into master Jul 13, 2017
@gemmaleigh gemmaleigh deleted the allow-release-deployment-to-fail branch July 13, 2017 13:33
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