Skip to content

Arduino support layer: several fixes#21039

Merged
maribu merged 4 commits intoRIOT-OS:masterfrom
mguetschow:arduino-fixes
Nov 26, 2024
Merged

Arduino support layer: several fixes#21039
maribu merged 4 commits intoRIOT-OS:masterfrom
mguetschow:arduino-fixes

Conversation

@mguetschow
Copy link
Copy Markdown
Contributor

Contribution description

Some minor fixes and enhancements for working with Arduino sketches in RIOT.

Testing procedure

Check that diff makes sense, should be no code breaking changes.

Some feedback on the explicit stdin would be nice. The semantics of this pseudomodule do not seem to be defined anywhere explicitly. Maybe @benpicco has some insights?

Issues/PRs references

Encountered while working on #20980

@mguetschow mguetschow requested a review from aabadie as a code owner November 25, 2024 15:27
@github-actions github-actions bot added Area: boards Area: Board ports Area: examples Area: Example Applications labels Nov 25, 2024
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 25, 2024
@maribu maribu enabled auto-merge November 25, 2024 16:07
@riot-ci
Copy link
Copy Markdown

riot-ci commented Nov 25, 2024

Murdock results

✔️ PASSED

6e61c1b examples/arduino_hello-world: this example needs stdin

Success Failures Total Runtime
10249 0 10249 17m:59s

Artifacts

@maribu maribu added this pull request to the merge queue Nov 25, 2024
Merged via the queue into RIOT-OS:master with commit c91aee0 Nov 26, 2024
@mguetschow mguetschow deleted the arduino-fixes branch November 26, 2024 12:01
@mguetschow
Copy link
Copy Markdown
Contributor Author

Thanks! :)

@MrKevinWeiss MrKevinWeiss added this to the Release 2025.01 milestone Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: boards Area: Board ports Area: examples Area: Example Applications CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants