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

CI Optimization #4009

Closed
microdev1 opened this issue Jan 16, 2021 · 0 comments · Fixed by #5922
Closed

CI Optimization #4009

microdev1 opened this issue Jan 16, 2021 · 0 comments · Fixed by #5922
Labels
enhancement tooling
Milestone
Long term

Comments

@microdev1
Copy link
Collaborator

Documenting some ways to optimize Build-CI runs :-

  • Deleting previously running CI tasks on a PR when a new CI task starts on the same PR.
  • If the changes are only in port specific sub-directory. Don't run test for other ports.
  • Running documentation build when changes are in the places that we know contain docs (e.g. shared-bindings directory).
@tannewt tannewt added this to the Long term milestone Jan 20, 2021
@tannewt tannewt mentioned this issue Sep 7, 2021
Merged
dhalbert pushed a commit that referenced this issue Sep 8, 2021
@tannewt
Selectively build boards based on changed files
a874043 
This uses a step output from the test job to set the build matrix
for the board build jobs. The built boards depends on which files
were changed.

* Changes contained within ports/*/boards/ will build each board.
* Changes contained within ports/* will build all boards from the
  port.
* All other file changes will build all boards.

All boards will be build for pushes in `adafruit/circuitpython` as
well.

A side-effect is that we no longer need to explicitly list the
boards to build. It will automatically update as new folders are
added.

Related to #4009
@microdev1 microdev1 mentioned this issue Sep 15, 2021
Merged
@microdev1 microdev1 linked a pull request Jan 25, 2022 that will close this issue
Automatically cancel superseded CI build of PR #5922
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement tooling
Projects
None yet
Milestone
Long term
Development

Successfully merging a pull request may close this issue.

Automatically cancel superseded CI build of PR
2 participants
@tannewt @microdev1