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.

Sign up for GitHub

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

Jump to bottom

Remove Heroku deploy from Travis jobs, update docs #566

Merged
merged 2 commits into from
Oct 4, 2017
Merged

Remove Heroku deploy from Travis jobs, update docs #566

merged 2 commits into from
Oct 4, 2017

Commits on Oct 4, 2017

  1. Remove Heroku deploy from Travis jobs 

    Heroku is now configured to deploy the latest-release branch to the govuk-elements-sass-release app when CI passes, which means Travis no longer needs to do this.
    @36degrees
    36degrees committed Oct 4, 2017
    Configuration menu
    Copy the full SHA
    203a07f View commit details
    Browse the repository at this point in the history

  2. Make Travis job config clearer 

    - Update the automatic deploy comment to make it clearer
- Remove `on: master` from the create-release job as this is the default for deploy providers and it’s confusing to have it on one and not the others
- Rename the ’create release’ stage to better reflect what it actually does (‘check for new release’)
- Re-word confusing comments to make it clearer how and when the deploy stage gets triggered
- Reduce the line lengths of the file by moving comments above sections
    @36degrees
    36degrees committed Oct 4, 2017
    Configuration menu
    Copy the full SHA
    986666e View commit details
    Browse the repository at this point in the history