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 support for Travis CI builds #45

Merged
merged 4 commits into from
Jul 26, 2018
Merged

Conversation

harrisonhjones
Copy link
Contributor

@harrisonhjones harrisonhjones commented Jul 24, 2018

This code change adds support (a config file) for Travis CI builds. It also adds a build badge to the README. Example of it building my fork.

To get this to work you will also need to:

  1. Head over to Travis CI and sign-in with your GitHub account
  2. Once signed in you should get a list of your repos. Locate go-alexa and select it.
  3. Locate the blue green "Activate repository" button. Press it and follow any additional steps to get it setup.
  4. Travis CI will automatically start building new pull requests and commits (this can be customized). Once this PR is merged all new code will be built (using go test) against go 1.8, 1.9, 1.10, 1.x, and "tip". The build badge on the README will reflect the current state of building master.

@rking788
Copy link
Contributor

Looks good.

@harrisonhjones
Copy link
Contributor Author

A last-minute thought: some projects have their badges next to the header in their README. Should we do the same here? I was thinking we could also add a badge link to our godoc and to the go report card but that should be in a separate PR.

@mikeflynn
Copy link
Owner

This is awesome. I fixed the conflict in the README and setup the account, so I'm merging this in.

(The go tip job is taking a while to complete, but for this PR I think it's OK to merge without it!)

@mikeflynn mikeflynn merged commit 58969f1 into mikeflynn:master Jul 26, 2018
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.

3 participants