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.

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

doc: boards: align main board image name with board name #79503

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

kartben
Copy link
Collaborator

@kartben kartben commented Oct 7, 2024

As per board porting guidelines, the board image should be named after the board.

Compliance check failure is a false positive -- one of the images is too large, but resizing it would be counter productive at this point (old version would still live in Git history).

@marekmatej
Copy link

LGTM

soburi
soburi previously approved these changes Oct 7, 2024
thaoluonguw
thaoluonguw previously approved these changes Oct 8, 2024
wmrsouza
wmrsouza previously approved these changes Oct 8, 2024
nashif
nashif previously approved these changes Oct 8, 2024
uLipe
uLipe previously approved these changes Oct 8, 2024
sylvioalves
sylvioalves previously approved these changes Oct 8, 2024
As per board porting guidelines, the board image should be named after
the board.
This commit implements the changes for boards found to be non-compliant.

Signed-off-by: Benjamin Cabé <[email protected]>
@kartben
Copy link
Collaborator Author

kartben commented Oct 10, 2024

hadn't noticed docs CI failure until now, sorry. Compliance check will still be failing and will need super-power merge rights.

@kartben
Copy link
Collaborator Author

kartben commented Oct 15, 2024

@sylvioalves please re-approve if you have a chance

@nashif nashif merged commit 4f4c7b9 into zephyrproject-rtos:main Oct 15, 2024
18 of 19 checks passed
@kartben kartben deleted the board_images_cleanup branch October 15, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: ESP32 Espressif ESP32 platform: Renesas RA Renesas Electronics Corporation, RA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants