Migrate from Travis to GitHub Actions #1526

crankyoldgit · 2021-07-07T07:19:01Z

Move any remaining Travis processes to GitHub Actions
Remove Travis files and references. (Excluding ReleaseNotes.md)
Change badges as well.
- Remove two defunct badges.
- Add a badge per GitHub Actions workflow.

These are being run as GitHub actions. Reduce stuff running on Travis For #1522

crankyoldgit · 2021-07-07T07:42:58Z

@NiKiZe Hold ya horses! I hadn't added you as a reviewer yet. I'm still working on this. ;-)
In order for the Github Actions etc to trigger/run I had to create a PR.
As I work to clean this up/improve etc, I'll be adding more and more. Some will work. Some will fail.

I'll let you know when it's ready to review! ;-)

NiKiZe · 2021-07-07T07:43:02Z

Is there any way to cache platformio compiles to reduce time spent? Seeing the output the same files are compiled multiple times?

crankyoldgit · 2021-07-07T07:49:53Z

Is there any way to cache platformio compiles to reduce time spent? Seeing the output the same files are compiled multiple times?

Yes, and it is set up to do that. If the compile/build options are the same, it does cache them.

However, a lot of the "duplication" is due to different compiler flags being used. e.g. A build with -D_IR_LOCALE_=en-AU, then a build with -D_IR_LOCALE_=de-DE. When that happens, it can't cache the files. :-(

crankyoldgit · 2021-07-07T07:54:37Z

Is there any way to cache platformio compiles to reduce time spent? Seeing the output the same files are compiled multiple times?

Yes, and it is set up to do that. If the compile/build options are the same, it does cache them.

However, a lot of the "duplication" is due to different compiler flags being used. e.g. A build with -D_IR_LOCALE_=en-AU, then a build with -D_IR_LOCALE_=de-DE. When that happens, it can't cache the files. :-(

Well, let's just say, that's what it was doing under Travis, I haven't got this Github Actions stuff sorted yet.

* Make a workflow for linters * PlatformIO workflow now does all the examples. - Try to improve caching. * Try to improve UnitTest speed by "-j 2" for Make. * Remove the moved stuff from Travis For #1526

* Remove moved checks from Travis * Cleanup & improvement to Library Linter workflow.

* Also more cleanup.

* Doxygen check moved to github workflow. * No more travis checks, so remove file.

* Move from Travis to the new GitHub Workflows. * Remove old non-updating badges. For #1522

* Cut & paste fail. Fixed. * Add to FR and DE files. For #1522

Turns out the status badges use these to name the badges. So pick ones we'd like to see in the badges. For #1522

NiKiZe · 2021-07-08T11:09:34Z

There is no risk of running out of free credits for these actions?

crankyoldgit · 2021-07-08T11:43:04Z

There is no risk of running out of free credits for these actions?

They don't seem to have a quota system, for the free/public tier. So ... dunno.

* Make the workflow `.yml` file names more consistent. * Remove references to Travis.

crankyoldgit · 2021-07-08T12:06:17Z

Okay @NiKiZe this PR is done I think. Feel free to review it. ;-)

crankyoldgit · 2021-07-08T12:24:46Z

There is no risk of running out of free credits for these actions?

Just double checked. Scroll down a fair bit on https://github.com/features/actions. It's "Free" for Open Source. ;-) <phew>

NiKiZe

Awesome!

README.md

.github/workflows/UnitTests.yml

NiKiZe · 2021-07-09T19:37:37Z

There is no risk of running out of free credits for these actions?

Just double checked. Scroll down a fair bit on https://github.com/features/actions. It's "Free" for Open Source. ;-)

It only mentions private repos (to my eyes) ... the Free part was

to big

NiKiZe · 2021-07-09T19:39:25Z

Any chance we can limit number of languages built? Or maybe build everything in order of language instead?

crankyoldgit · 2021-07-09T23:35:50Z

Any chance we can limit number of languages built? Or maybe build everything in order of language instead?

Agreed. We need to do something better.
I'd prefer if they were only run/compiled when the locale stuff changed our for a release.

I don't know how to enact that yet.

- Caught in code review.

_v2.7.20 (20210828)_ **[Bug Fixes]** - Make `strToSwingH()` match "Right Max" (#1550 #1551) **[Features]** - Experimental Bose remote support (#1579) - Added MitsubishiAC VaneLeft (#1572 #1576) - HAIER_AC176: Add experimental detailed support (#1480 #1571) - Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568) - Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507) - SamsungAc: Use `sendExtended()` going forward. (#1484 #1562) - SamsungAc: Redo/fix checksum calculations. (#1538 #1554) - LG: Add support for `AKB73757604` model (#1531 #1545) - Daikin176: Add support for Unit Id. (#1543 #1544) - Daikin2: Add support for Humidity setting/operation. (#1535 #1540) - TCL112AC: Add support for quiet/mute setting. (#1528 #1529) - LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530) - Add Mitsubishi AC "fan only" mode (#1527) **[Misc]** - Fix pylint issues due to pylint update. (#1569 #1570) - DAIKIN216: Update supported models. (#1552 #1567) - IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541) - Reduce memory fragmentation cause by String usage. (#1493 #1536) - Refactor `decodeMitsubishiAC()` (#1523 #1532) - Fix incorrect comment. - Migrate from Travis to GitHub Actions (#1522 #1526) - Documentation update with additional supported Panasonic AC models (#1525)

_v2.7.20 (20210828)_ **[Bug Fixes]** - Make `strToSwingH()` match "Right Max" (#1550 #1551) **[Features]** - Experimental Bose remote support (#1579) - Added MitsubishiAC VaneLeft (#1572 #1576) - HAIER_AC176: Add experimental detailed support (#1480 #1571) - Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568) - Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507) - SamsungAc: Use `sendExtended()` going forward. (#1484 #1562) - SamsungAc: Redo/fix checksum calculations. (#1538 #1554) - LG: Add support for `AKB73757604` model (#1531 #1545) - Daikin176: Add support for Unit Id. (#1543 #1544) - Daikin2: Add support for Humidity setting/operation. (#1535 #1540) - TCL112AC: Add support for quiet/mute setting. (#1528 #1529) - LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530) - Add Mitsubishi AC "fan only" mode (#1527) **[Misc]** - Change when some github workflows run (#1583) - Add/update supported device info (#1580 #1581 #1585) - Fix pylint issues due to pylint update. (#1569 #1570) - DAIKIN216: Update supported models. (#1552 #1567) - IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541) - Reduce memory fragmentation cause by String usage. (#1493 #1536) - Refactor `decodeMitsubishiAC()` (#1523 #1532) - Fix incorrect comment. - Migrate from Travis to GitHub Actions (#1522 #1526) - Documentation update with additional supported Panasonic AC models (#1525)

## _v2.7.20 (20210828)_ **[Bug Fixes]** - Make `strToSwingH()` match "Right Max" (#1550 #1551) **[Features]** - Experimental Bose remote support (#1579) - Added MitsubishiAC VaneLeft (#1572 #1576) - HAIER_AC176: Add experimental detailed support (#1480 #1571) - Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568) - Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507) - SamsungAc: Use `sendExtended()` going forward. (#1484 #1562) - SamsungAc: Redo/fix checksum calculations. (#1538 #1554) - LG: Add support for `AKB73757604` model (#1531 #1545) - Daikin176: Add support for Unit Id. (#1543 #1544) - Daikin2: Add support for Humidity setting/operation. (#1535 #1540) - TCL112AC: Add support for quiet/mute setting. (#1528 #1529) - LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530) - Add Mitsubishi AC "fan only" mode (#1527) **[Misc]** - Change when some github workflows run (#1583) - Add/update supported device info (#1580 #1581 #1585) - Fix pylint issues due to pylint update. (#1569 #1570) - DAIKIN216: Update supported models. (#1552 #1567) - IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541) - Reduce memory fragmentation cause by String usage. (#1493 #1536) - Refactor `decodeMitsubishiAC()` (#1523 #1532) - Fix incorrect comment. - Migrate from Travis to GitHub Actions (#1522 #1526) - Documentation update with additional supported Panasonic AC models (#1525)

crankyoldgit added 2 commits July 7, 2021 16:56

Don't run unit tests under Travis.

0b4f8cc

These are being run as GitHub actions. Reduce stuff running on Travis For #1522

Attempt to build some examples using Github Actions & PlatformIO

e3397d5

NiKiZe approved these changes Jul 7, 2021

View reviewed changes

crankyoldgit added 14 commits July 7, 2021 20:14

More workflows & changes

83db277

* Make a workflow for linters * PlatformIO workflow now does all the examples. - Try to improve caching. * Try to improve UnitTest speed by "-j 2" for Make. * Remove the moved stuff from Travis For #1526

Fix some python linter errors

5a910e9

Another attempt to fix linter error

92c389a

Try to fix cpplint

861c24f

Move the Misc checks to GitHub Actions.

57ad14c

* Remove moved checks from Travis * Cleanup & improvement to Library Linter workflow.

Fix errors library manager workflow

34911f7

* Also more cleanup.

Another fix.

d489a55

Fix error in name

ec6b18b

Move doxygen check to workflow

85ec8cb

* Doxygen check moved to github workflow. * No more travis checks, so remove file.

Update status badges.

de215d2

* Move from Travis to the new GitHub Workflows. * Remove old non-updating badges. For #1522

Fix badge issue

9dd8293

* Cut & paste fail. Fixed. * Add to FR and DE files. For #1522

Use better names in the Workflows

efb814c

Turns out the status badges use these to name the badges. So pick ones we'd like to see in the badges. For #1522

Make the new status badges be links to the relevent actions.

6433ddb

Hopefully with the correct URL this time.

d4270c4

Rename workflow files & remove Travis references.

90984e4

* Make the workflow `.yml` file names more consistent. * Remove references to Travis.

crankyoldgit changed the title ~~Migrate more tasks from Travis to GitHub Actions~~ Migrate from Travis to GitHub Actions Jul 8, 2021

crankyoldgit requested a review from NiKiZe July 8, 2021 12:05

NiKiZe approved these changes Jul 9, 2021

View reviewed changes

README.md Show resolved Hide resolved

.github/workflows/UnitTests.yml Show resolved Hide resolved

Add "-j 2" for google test suite

d62bd5b

- Caught in code review.

crankyoldgit merged commit c71a150 into master Jul 10, 2021

crankyoldgit deleted the ci-changes branch July 21, 2021 13:29

crankyoldgit mentioned this pull request Aug 28, 2021

v2.7.20 release #1582

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate from Travis to GitHub Actions #1526

Migrate from Travis to GitHub Actions #1526

crankyoldgit commented Jul 7, 2021 •

edited

Loading

crankyoldgit commented Jul 7, 2021

NiKiZe commented Jul 7, 2021

crankyoldgit commented Jul 7, 2021

crankyoldgit commented Jul 7, 2021

NiKiZe commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

NiKiZe left a comment

NiKiZe commented Jul 9, 2021

NiKiZe commented Jul 9, 2021

crankyoldgit commented Jul 9, 2021

Migrate from Travis to GitHub Actions #1526

Migrate from Travis to GitHub Actions #1526

Conversation

crankyoldgit commented Jul 7, 2021 • edited Loading

crankyoldgit commented Jul 7, 2021

NiKiZe commented Jul 7, 2021

crankyoldgit commented Jul 7, 2021

crankyoldgit commented Jul 7, 2021

NiKiZe commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

crankyoldgit commented Jul 8, 2021

NiKiZe left a comment

Choose a reason for hiding this comment

NiKiZe commented Jul 9, 2021

to big

NiKiZe commented Jul 9, 2021

crankyoldgit commented Jul 9, 2021

crankyoldgit commented Jul 7, 2021 •

edited

Loading