arduino / arduino/ArduinoCore-samd

Builds broken in master branch (1.8.11)?!

Open
#616 6 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
502
Forks
740
PR merge metrics
No merged PRs in 30d

Description

Is something misconfigured for the CI builds?

It appears that all PRs since December 24 have failed to build (via CI Actions).

On December 24th, three PRs (#3, #586, #588) were merged, and then the version was updated to 1.8.11. Each of those merges has failed the CI build (but logs are gone now).

Please fix so PRs can have some sanity checks applied, and be considered for merging again!

Potentially affected PRs include:

* #613 - Fix compiler warning from useless volatile
* #612 - Don't spill arduino namespace in headers
* #610 - fix: address warning
* #605 - Update Wire peripheral handler
* #602 - Update HID.h with report types
* #600 - Add a weak attribute to serial IRQ allowing real time communication lib.
* #592 - Improve on bootloader security when Security Bit is set
* #591 - Fix I2C Master hang when communicating with bad I2C slaves.
* #590 - Slave Wire: fix TwoWire::onService() to handle repeated start

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by inspecting the repository's CI Actions configuration and the current master-branch build setup; the issue says the historical logs are gone. Compare the configuration with the merged changes and reproduce a failing pull-request build. Done means pull requests receive successful CI sanity checks again.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, github-actions
Domain
build-system, ci-cd
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.