search

AmboVent-1690-108 / AmboVent

AmboVent 1690.108
The Unlicense
206 stars 69 forks source link

Travis CI integration example #42

Open labeneator opened 4 years ago

labeneator commented 4 years ago

A quick example on how to integrate travis build status.

labeneator commented 4 years ago

Fixes #25. Should also reference #40

ElectricRCAircraftGuy commented 4 years ago

@labeneator , this looks pretty useful. Thanks. I'd like to dig into this when I get the chance. Have you done this before or did you just learn how to do it?

labeneator commented 4 years ago

Just figured it out in a couple of hours :)

On Mon, 20 Apr 2020, 11:52 Gabriel Staples, notifications@github.com wrote:

@labeneator https://github.com/labeneator , this looks pretty useful. Thanks. I'd like to dig into this when I get the chance. Have you done this before or did you just learn how to do it?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AmboVent-1690-108/AmboVent/pull/42#issuecomment-616406915, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFU37DFNC3V7FWWIM47FLLRNQENTANCNFSM4MIAAMRQ .

ElectricRCAircraftGuy commented 4 years ago

@labeneator Nice! Can you post all the links you used to figure it out please? I'll take a closer look at the links and the PR one of these evenings. Also, is there any way to test it before merging it, or must it be merged first to be able to test it?

labeneator commented 4 years ago

Sorry for the late response.

  1. Started looking up for Continuous integration options for github which led me to adafruit.
  2. I remembered that travis-ci is well integrated with github. I checked their integration options for arduino
  3. I had never used platform.io, however the examples given were intuitive and that, led to this PR.

The next steps would be to have the code integrated with unit-test libraries for arduino. A quick google leads to these options, that need to be investigated for suitability:

  1. https://maker.pro/arduino/tutorial/using-unit-test-frameworks-with-arduino
  2. https://www.thecoderscorner.com/electronics/microcontrollers/embedded-testing/getting-started-unittesting-arduino/
  3. https://docs.platformio.org/en/latest/plus/unit-testing.html
  4. https://github.com/bxparks/AUnit
  5. https://github.com/mmurdoch/arduinounit
labeneator commented 4 years ago

Also, is there any way to test it before merging it, or must it be merged first to be able to test it?

I think the best thing to do is merge it to a different branch for testing on your repo. You will require to setup a travis-ci account for ambovent. Once you are happy with the setup, you can merge back to master

labeneator commented 4 years ago

If you create a travis-ci branch on your repo, I can target this PR @ that branch.