adafruit / ci-arduino

A script that will install all of the common dependencies for testing Arduino library builds using Github Actions
MIT License
115 stars 73 forks source link

Dont warn that learn isn't marked as an arduino library #146

Closed jepler closed 10 months ago

jepler commented 1 year ago

I noticed that in the learn repo, ci-arduino warns that "arduino-library is not found in this repo topics." Change the way the repos are selected to run this check so that this warning won't be printed there.

hathach commented 1 year ago

FYI: repo that tags as "arduino-libraries" is for rebuild repo when there is a new BSP released e.g SAMD, nrf52 or arduino-esp32

https://github.com/adafruit/arduino-board-index/blob/gh-pages/.github/workflows/trigger_libraries.yml

for example https://github.com/adafruit/Adafruit_TinyUSB_Arduino/actions/runs/4210118239 . Maybe consider to tag the learn guide as 'arduino-libraries' as well for it to be rebuilt.

ladyada commented 10 months ago

@hathach i did both!