Skip to content

Update CI workflow to correctly install ArduinoCore-API in arduino:mbed core #46

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 2 commits into from
Dec 15, 2020
Merged

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Dec 15, 2020

Recent changes to the Arduino Mbed OS Boards platform core's handling of the arduino/ArduinoCore-API repository resulted in the need to adjus how the "Compile Examples" CI workflow installs it for those boards.


The symlink for ArduinoCore-API in the Mbed OS Boards core library has been removed (arduino/ArduinoCore-mbed@ca397e8). This resulted in the step of the "Compile Examples" CI workflow that removes the now non-existent symlink to fail.


Previously, the namespace_arduino branch of ArduinoCore-API was used in the core library of the development version of
the Mbed OS Boards platform used in the "Compile Examples" CI workflow. The default branch of ArduinoCore-API is now the
latest and greatest version, so the namespace_arduino branch should no longer be used for the Mbed OS Boards platform's core library (arduino/ArduinoCore-mbed@3c175d7).

The symlink for ArduinoCore-API in the Mbed OS Boards core library has been removed. This resulted in the step of the
"Compile Examples" CI workflow that removes the now non-existent symlink to fail.
Previously, the `namespace_arduino` branch of `arduino/ArduinoCore-API` was used in the core library of the development
version of the Mbed OS Boards platform used in the "Compile Examples" CI workflow. The default branch of
`arduino/ArduinoCore-API` is now the latest and greatest version, so the `namespace_arduino` branch should no longer be
used for the Mbed OS Boards platform's core library.
@github-actions
Copy link

Memory usage change @ 3b0e185

Board flash % RAM for global variables %
arduino:mbed:envie_m4 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:mbed:envie_m7 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkr1000 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrgsm1400 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrnb1500 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrwan1300 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrwan1310 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrwifi1010 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:nano_33_iot 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
esp8266:esp8266:huzzah 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board examples/ConnectionHandlerDemo
flash
% examples/ConnectionHandlerDemo
RAM for global variables
%
arduino:mbed:envie_m4 0 0.0 0 0.0
arduino:mbed:envie_m7 0 0.0 0 0.0
arduino:samd:mkr1000 0 0.0 0 0.0
arduino:samd:mkrgsm1400 0 0.0 0 0.0
arduino:samd:mkrnb1500 0 0.0 0 0.0
arduino:samd:mkrwan1300 0 0.0 0 0.0
arduino:samd:mkrwan1310 0 0.0 0 0.0
arduino:samd:mkrwifi1010 0 0.0 0 0.0
arduino:samd:nano_33_iot 0 0.0 0 0.0
esp8266:esp8266:huzzah 0 0.0 0 0.0
Click for full report CSV
Board,examples/ConnectionHandlerDemo<br>flash,%,examples/ConnectionHandlerDemo<br>RAM for global variables,%
arduino:mbed:envie_m4,0,0.0,0,0.0
arduino:mbed:envie_m7,0,0.0,0,0.0
arduino:samd:mkr1000,0,0.0,0,0.0
arduino:samd:mkrgsm1400,0,0.0,0,0.0
arduino:samd:mkrnb1500,0,0.0,0,0.0
arduino:samd:mkrwan1300,0,0.0,0,0.0
arduino:samd:mkrwan1310,0,0.0,0,0.0
arduino:samd:mkrwifi1010,0,0.0,0,0.0
arduino:samd:nano_33_iot,0,0.0,0,0.0
esp8266:esp8266:huzzah,0,0.0,0,0.0

Copy link
Contributor

@aentinger aentinger left a comment

Choose a reason for hiding this comment

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

Thank you @per1234 🚀

@aentinger aentinger merged commit dbfb0e2 into arduino-libraries:master Dec 15, 2020
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