Skip to content

Add Travis Continuous Integration to the repo - #86

Open
peternewman wants to merge 10 commits into
CNMAT:masterfrom
peternewman:patch-2
Open

Add Travis Continuous Integration to the repo#86
peternewman wants to merge 10 commits into
CNMAT:masterfrom
peternewman:patch-2

Conversation

@peternewman

@peternewman peternewman commented Jun 28, 2018

Copy link
Copy Markdown
Contributor

To catch careless typos etc.

You'll need to enable Travis on the repo (https://travis-ci.com is now the recommended option), and I'd then suggest you protect the branch in GitHub so you can only merge if Travis passes.

Currently this is just testing against the some Teensy version, we could test against a lot more variations if it's worth it (many examples, many platforms):
http://platformio.org/boards

We could also enable a C++ linter if you have a preferred one, or use cpplint.py from Google?

@peternewman

Copy link
Copy Markdown
Contributor Author

It's already found a pile of typos:
https://travis-ci.com/peternewman/OSC/jobs/131740761

@peternewman

Copy link
Copy Markdown
Contributor Author

Typos all fixed.

@peternewman

Copy link
Copy Markdown
Contributor Author

This now all passes apart from these two, which looks like genuine failures (on Teensy 3.5/6):
https://travis-ci.com/peternewman/OSC/jobs/134879442#L609
https://travis-ci.com/peternewman/OSC/jobs/134879447#L609

@peternewman

Copy link
Copy Markdown
Contributor Author

Hi @adrianfreed,

What's your feeling on this? Should Teensy 3.5/6 be supported? Are you expecting the UDPSendBundlewithTimeTag examples to work on those platforms?

Should I mark those as expected failures for now so the bulk of Travis can be merged in and the spelling fixes?

@peternewman peternewman changed the title Add Travis Continuous Integration to the repo Add Travis Continuous Integration to the repo and fix found spelling mistakes Sep 25, 2018
@peternewman peternewman mentioned this pull request Oct 1, 2018
@peternewman peternewman changed the title Add Travis Continuous Integration to the repo and fix found spelling mistakes Add Travis Continuous Integration to the repo Oct 29, 2018

@adrianfreed adrianfreed left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

too intrusive a change

adrianfreed added a commit that referenced this pull request Aug 4, 2026
…ists

peternewman proposed Travis CI in PR #86 in 2018. The intent was right and
is honoured here; the mechanism is not, and could not be merged:

 - travis-ci.com no longer builds open-source repositories. The badge that
   PR adds to the README fetches today and renders a permanent grey
   "build unknown".
 - Its config uses dist: trusty, sudo: false and group: edge, all retired
   Travis concepts, and installs lintian from a Trusty PPA.
 - Its lint task downloads cpplint.py from a URL that now returns 404, and
   the script runs under bash -ex, so the job would die on that line. No
   matrix entry sets TASK=lint, so it has never run.

What lands instead runs on every push and pull request:

 - the host suite under AddressSanitizer and UBSan, which needs no board
   and no Arduino toolchain
 - the Web Serial generator drift check and codec tests
 - example compiles, one job per core so a failure names the platform,
   across arduino:avr (uno, leonardo, esplora), arduino:samd, esp32 and
   rp2040

Each core builds only the sketches it can, since the ESP8266, WiFi,
Ethernet and board-specific examples need libraries or hardware a given
core does not have. Each compile gets its own --build-path: arduino-cli
keys its default build cache on the sketch path alone, so jobs sharing a
runner would otherwise stomp each other's objects.

Both make targets verified locally exactly as CI invokes them:
make -C test/host asan, and make -C extras/webserial all (39 passed).

Co-Authored-By: peternewman <peternewman@users.noreply.github.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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