Skip to content

macos CI: test on push to release branches only#1672

Merged
scpeters merged 1 commit into
mainfrom
scpeters/ci_macos_on_push_release
Jun 25, 2026
Merged

macos CI: test on push to release branches only#1672
scpeters merged 1 commit into
mainfrom
scpeters/ci_macos_on_push_release

Conversation

@scpeters

@scpeters scpeters commented Jun 24, 2026

Copy link
Copy Markdown
Member

🦟 Bug fix

Reduces macOS job triggering

Summary

Currently our GitHub workflow for macOS CI runs on every push to a branch, whereas our other CI workflows only run on pushes to release branches. This updates the macOS job triggering to match our Ubuntu CI triggering.

The impact of this change is to reduce the number of CI minutes used and to prevent triggering of duplicate CI jobs (from push and pull_request), potentially reducing the time spent waiting for CI to complete.

Backport Policy

  • This is safe to backport to the following versions:
    • Jetty
    • Ionic
    • Harmonic
    • Fortress
  • This should not be backported
  • I am not sure
  • Other (fill in yourself)

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
@scpeters scpeters marked this pull request as ready for review June 24, 2026 00:28
@scpeters scpeters requested a review from azeey as a code owner June 24, 2026 00:28
@github-project-automation github-project-automation Bot moved this from Inbox to In review in Core development Jun 25, 2026
@scpeters scpeters merged commit ed54622 into main Jun 25, 2026
17 checks passed
@scpeters scpeters deleted the scpeters/ci_macos_on_push_release branch June 25, 2026 15:43
@github-project-automation github-project-automation Bot moved this from In review to Done in Core development Jun 25, 2026
@scpeters

Copy link
Copy Markdown
Member Author

https://github.com/Mergifyio backport sdf16

@mergify

mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

backport sdf16

✅ Backports have been created

Details

scpeters added a commit that referenced this pull request Jun 25, 2026
Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
(cherry picked from commit ed54622)
@scpeters

Copy link
Copy Markdown
Member Author

https://github.com/Mergifyio backport sdf15

@mergify

mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

backport sdf15

✅ Backports have been created

Details

scpeters added a commit that referenced this pull request Jun 26, 2026
Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
(cherry picked from commit ed54622)
@scpeters

Copy link
Copy Markdown
Member Author

https://github.com/Mergifyio backport sdf14 sdf12

@mergify

mergify Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

backport sdf14 sdf12

✅ Backports have been created

Details

scpeters added a commit that referenced this pull request Jun 26, 2026
Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
(cherry picked from commit ed54622)
scpeters added a commit that referenced this pull request Jun 26, 2026
Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
(cherry picked from commit ed54622)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants