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

Add continuous integration tooling #44

Merged
merged 1 commit into from
Apr 16, 2015
Merged

Conversation

alukach
Copy link
Member

@alukach alukach commented Apr 15, 2015

After a successful build, Travis will run .travis/deploy-to-gh-pages.sh. This script will check to see that the build took place on the RELEASE_BRANCH (configured as a variable within the Travis configuration file), and if it has grunt gh-pages is ran, copying the repo (including ignored files/dirs, excluding dot files/dirs) to the gh-pages branch. This requires that Travis have a [Github OAuth token] added as a variable (this is done as a hidden variable on the Travis website, rather than within the config file). I've gone ahead and set Travis up with a token.

Currently, RELEASE_BRANCH is set as master, meaning any commits to master will be deployed. If this proves to be a source of error, we can adjust as necessary.

This addresses #38

@mapsam
Copy link
Member

mapsam commented Apr 16, 2015

@alukach this is really terrific. So many freaking things happening I can't keep my head straight but I'm trusting you 100% on this one. Sounds like we should wait until @aaronr finishes up #45 before this merges in. But otherwise 👍

@alukach
Copy link
Member Author

alukach commented Apr 16, 2015

@svmatthews Do you think so? I feel like this is independent of #45 (or maybe a requirement for #45). This has functionality on its own. I'm also a fan of keeping as few PRs open at any given time, helps make merges a bit easier. Open to merging it?

alukach added a commit that referenced this pull request Apr 16, 2015
@alukach alukach merged commit 31f4975 into master Apr 16, 2015
@mapsam
Copy link
Member

mapsam commented Apr 16, 2015

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants