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

CI: Skip the CI Builds that don't match the Board Label #86

Closed
wants to merge 1 commit into from

Conversation

lupyuen
Copy link

@lupyuen lupyuen commented Oct 10, 2024

Summary

This PR extends the CI Build Rules to eliminate more unnecessary builds. We use the Board Label for PRs:

  • Previously: "Arch: arm" will build only arm-01 to arm-14
  • Now: "Board: arm" will also build arm-01 to arm-14
  • This applies only to Simple PRs: One Arch Label + One Board Label + One Size Label, like "Arch: arm, Board: arm, Size: L"
  • If Arch Label and Board Label are both present: They must be the same. Otherwise All Targets shall be built.
  • Works with Arm32, Arm64, RISC-V, Simulator, x86_64 and Xtensa

The updated code is explained here: apache#13775

Impact

Simple PRs targeting One Single Board and/or Arch shall complete the CI Build a lot quicker now.

The Updated CI Workflow shall be synced to nuttx-apps repo in the next PR.

Testing

We tested by creating PRs with Arch and Board Labels:

@lupyuen lupyuen changed the title Update CI: Skip the CI Builds that don't match the Board Label Oct 10, 2024
This PR extends the CI Build Rules to eliminate more unnecessary builds. We use the Board Label for PRs:
- Previously: "Arch: arm" will build only `arm-01` to `arm-14`
- Now: "Board: arm" will also build `arm-01` to `arm-14`
- This applies only to Simple PRs: One Arch Label + One Board Label + One Size Label, like "Arch: arm, Board: arm, Size: L"
- If Arch Label and Board Label are both present: They must be the same. Otherwise All Targets shall be built.
- Works with Arm32, Arm64, RISC-V, Simulator, x86_64 and Xtensa

The updated code is explained here: apache#13775
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant