Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-55 Pantheon Travis automatic deploys #71

Merged
merged 77 commits into from
Mar 11, 2016

Conversation

pvhee
Copy link
Contributor

@pvhee pvhee commented Mar 3, 2016

Fixes https://jira.comicrelief.com/browse/PLAT-55

Changes proposed in this pull request

  • Deploy Pantheon site every time a PR is merged into develop
  • Compile SASS inside Travis, then commit it to Pantheon master. This also helps for tests since previously no compiled sass was present.
  • Reduce git history - this makes git checkout much faster and thus Travis runs faster

@@ -10,6 +10,13 @@ sudo: false
php:
- 5.6

# No need for git history here
git:
depth: 3
Copy link
Contributor

Choose a reason for hiding this comment

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

@pvhee I would use depth: 1 and also the option --single-branch because you only want to test your last commit and you don't need other branches

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point @Saphyel - will address

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't think we can do --single-branch with travis though, from travis-ci/travis-ci#4806

@pvhee
Copy link
Contributor Author

pvhee commented Mar 11, 2016

@Saphyel care to have another look here? I refactored this a bunch and when travis goes green this is fine for me to get merged in!

Saphyel added a commit that referenced this pull request Mar 11, 2016
@Saphyel Saphyel merged commit a65c4c3 into develop Mar 11, 2016
@Saphyel Saphyel deleted the feature/PLAT-55_pantheon_travis branch March 11, 2016 13:44
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants