Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tests/external_boards: add esp32s3-ci board for optional modules #18422

Merged
merged 1 commit into from
Aug 27, 2022
Merged

tests/external_boards: add esp32s3-ci board for optional modules #18422

merged 1 commit into from
Aug 27, 2022

Conversation

gschorcht
Copy link
Contributor

@gschorcht gschorcht commented Aug 9, 2022 •
edited
Loading

Contribution description

This PR adds an external board definition to cover optional ESP modules in CI compilation test for ESP32-S3.

The board definition corresponds to the existing external board definition for ESP32 tests/external_board_dirs/esp-ci-boards.

Testing procedure

Green CI

Issues/PRs references

Depends on PR #18421

@github-actions github-actions bot added Area: CI Area: Continuous Integration of RIOT components Area: Kconfig Area: Kconfig integration Area: tests Area: tests and testing framework labels Aug 9, 2022
@gschorcht gschorcht added Type: new feature The issue requests / The PR implemements a new feature for RIOT State: waiting for other PR State: The PR requires another PR to be merged first Area: boards Area: Board ports Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Aug 9, 2022
@gschorcht gschorcht force-pushed the cpu/esp32s3/esp32s3-ci_board branch from ee028d8 to fe7ecef Compare August 22, 2022 15:41
@github-actions github-actions bot removed Area: boards Area: Board ports Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Aug 22, 2022
@gschorcht gschorcht requested a review from benpicco August 22, 2022 15:41
@gschorcht gschorcht added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed State: waiting for other PR State: The PR requires another PR to be merged first labels Aug 22, 2022
@gschorcht gschorcht merged commit c5e72e1 into RIOT-OS:master Aug 27, 2022
@maribu maribu added this to the Release 2022.10 milestone Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benpicco benpicco benpicco approved these changes

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

@smlng smlng Awaiting requested review from smlng

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

Assignees
No one assigned
Labels
Area: CI Area: Continuous Integration of RIOT components Area: Kconfig Area: Kconfig integration Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Milestone
Release 2022.10
Development

Successfully merging this pull request may close these issues.
3 participants
@gschorcht @benpicco @maribu