milesburton / Arduino-Temperature-Control-Library

Arduino Temperature Library
https://www.milesburton.com/w/index.php/Dallas_Temperature_Control_Library
974 stars 486 forks source link

add Arduino-CI + prepare unit tests #192

Closed RobTillaart closed 3 years ago

RobTillaart commented 3 years ago

Based upon - https://github.com/marketplace/actions/arduino_ci

Goal The goal of continuous integration and unit testing is to have a set of automated actions to improve the quality and consistency of the code and its interface. To reach this goal two major steps exist

  1. Automatic compilation of all examples on different platforms this PR
  2. Automatic unit tests disabled

The automatic build of examples shows

The automatic compilation is done for (to be expanded in the future)

Unit Tests disabled The unit test is disabled by altering its extension. The reason is that the mocking / including of the complete Arduino environment is not complete yet. An issue has been raised for this and equivalent libraries - https://github.com/Arduino-CI/arduino_ci/issues/251

Notes

image (these are just an example)

RobTillaart commented 3 years ago

update This PR solves #182 a new issue for updating the unit tests could be made.

milesburton commented 3 years ago

Let me know if you're ready to merge, looks like a good initial addition

RobTillaart commented 3 years ago

I'm ready, already preparing the next step today, the Arduino-lint workflow . I have about 40 fails in 120 libraries so I'm busy for a few days 😎 (also means ~80 are OK)

RobTillaart commented 3 years ago

Got some first feedback from the Arduino-lint action for the Dallas Temp lib See - https://github.com/RobTillaart/Arduino-Temperature-Control-Library/runs/2686896760?check_suite_focus=true

RobTillaart commented 3 years ago

Created an issue (question) about the filename - https://github.com/arduino/arduino-lint/issues/178