-
Notifications
You must be signed in to change notification settings - Fork 83
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
Setup travis build. #28
Comments
@tehn : looks like an admin needs to flip the bit on travis for teletype: https://travis-ci.org/profile/monome Could you? |
flipped On Wed, Apr 6, 2016 at 5:35 PM, Phil Quitslund notifications@github.com
|
i must say, i am completely baffled by the contents of .travis.yml On Wed, Apr 6, 2016 at 5:35 PM, Phil Quitslund notifications@github.com
|
Ha! I blame @samdoshi. |
ah, it makes more sense now that i actually read about travis. so right now we're just doing a compilation check, correct? On Wed, Apr 6, 2016 at 6:23 PM, Phil Quitslund notifications@github.com
|
Right. Once we have tests we should add them. Down the road we could add linting or some other kind of presubmit checks. Note that pull requests trigger a build which means you can see if a PR will break the build before accepting it. For example: Which is pretty neat! |
looks good, thanks! |
You and me both. Should we get https://github.com/samdoshi/avr32-toolchain-linux moved over to the 'monome' org too? |
👍 Make sense to me! |
A few bits:
.travis.ym;
appropriately --- @samdoshi has proved this out (samdoshi@0a51dd8), so this shouldn't be too hardREADME
The text was updated successfully, but these errors were encountered: