Skip to content

Add configuration for Travis CI#10

Merged
amodm merged 3 commits intoamodm:masterfrom
ErichDonGubler:ci
Jun 22, 2018
Merged

Add configuration for Travis CI#10
amodm merged 3 commits intoamodm:masterfrom
ErichDonGubler:ci

Conversation

@ErichDonGubler
Copy link
Copy Markdown
Contributor

@ErichDonGubler ErichDonGubler commented Jun 20, 2018

This is my strong opinion and I hope it doesn't come across as an imposition: CI like this would have helped you realize that things were broken on Linux before cutting a 0.3 release! This is definitely YOUR crate, though, and it's your call whether or not this is appropriate for the development you do here. That's not a wrong decision either!

This Travis config is borrowed from the great-ci repo. Some functionalities you may not actually want even if you decide you want to accept this form of CI generally might be:

  • Code coverage analysis
  • Putting docs into a gh-pages branch

...which can easily be removed, no worries! 😄

This PR has a soft dependency on PR #9 -- N.B that CI will break without that pass. :)

@amodm amodm merged commit e6d7f6f into amodm:master Jun 22, 2018
@amodm
Copy link
Copy Markdown
Owner

amodm commented Jun 22, 2018

Thanks @ErichDonGubler. Much appreciated. My dev env is OSX, so this is particularly helpful.

@ErichDonGubler ErichDonGubler deleted the ci branch June 22, 2018 14:35
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