Skip to content

Travis CI Support #31

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

Merged
merged 10 commits into from
Oct 15, 2018
Merged

Travis CI Support #31

merged 10 commits into from
Oct 15, 2018

Conversation

dmadison
Copy link
Owner

Adds Travis CI support. Whenever a new commit is pushed (either by me or from a pull request), Travis will test its validity by building all of the library examples in a virtual instance.

Currently tested boards:

  • Arduino Uno
  • Teensy LC

These are the two boards the library was designed around, and all examples are compiled for both boards (sans the dual I²C bus one). In the future it might be wise to also unit test other boards like the Leonardo or the ESP8266, but for now this will suffice. (Don't want to needlessly tax Travis CI's free service, either.)

Currently Travis is configured to download Arduino 1.8.1 and Teensyduino 1.44. I'm using a slightly older version of the Arduino IDE because more recent ones (>1.8.1, e.g. 1.8.6 or 1.8.7) apparently have issues with SocketListener warnings that pollute the build log (see per1234/arduino-ci-script#1). For now this seems to work fine; if this issue gets fixed or I start running into compatibility issues I'll bump the version.

Install Arduino and builds all examples for both the Uno and the Teensy LC.
Forgot the prefix decimal when copying. D'oh!
Easier than listing every example name.
Perks of not having a setup to test the YAML parsing
Making the build log easier to read with some separation.
Prevents checking board names against a list
@dmadison dmadison merged commit c009758 into master Oct 15, 2018
@dmadison dmadison deleted the travis-ci branch October 15, 2018 15:02
@dmadison
Copy link
Owner Author

Hah, and I merged the PR before waiting for the PR-specific build to start, so the PR build failed because I had already merged it and deleted the branch. Well that's ironic!

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.

1 participant