Skip to content

ci: extend windows-latest + macos-latest push triggers to release/v*#297

Merged
ChrisonSimtian merged 1 commit into
mainfrom
ci/extend-push-triggers-to-release-branches
May 29, 2026
Merged

ci: extend windows-latest + macos-latest push triggers to release/v*#297
ChrisonSimtian merged 1 commit into
mainfrom
ci/extend-push-triggers-to-release-branches

Conversation

@ChrisonSimtian

Copy link
Copy Markdown
Collaborator

Closes #293. Parity follow-up to #291, which extended PR-validation triggers to release/v* but deliberately left push triggers alone.

Why

After #291 + #292, PRs against release/v* get ubuntu-latest. But post-merge cross-platform validation (windows-latest + macos-latest) still only fires on push to main. A hotfix landing on release/v11 with a subtle Windows or macOS regression wouldn't be caught until someone attempts a release and the cross-platform CI in release.yml's Test step fails — higher-stakes path, harder to fix forward.

What changes

  • build/Build.CI.GitHubActions.csOnPushBranches on both windows-latest and macos-latest [GitHubActions] entries now reads new[] { MainBranch, ReleaseBranchPattern }. ReleaseBranchPattern was added in ci: extend PR-validation triggers to release/v* branches #291 alongside MainBranch.
  • .github/workflows/windows-latest.ymlpush.branches adds 'release/v*'.
  • .github/workflows/macos-latest.yml — same.

YAML hand-edited to match the source — next time someone runs dotnet fallout --generate-configuration GitHubActions_windows-latest GitHubActions_macos-latest, the regen produces identical output.

Effect

A hotfix merged to release/v11 now fires Windows + macOS post-merge validation. Cross-platform regressions surface immediately as a red commit on the release branch, not later during a release attempt.

Out of scope

  • Pull-request triggers on windows/macos. Those stay PR-only on main. The current PR gate (ubuntu-latest only) is a deliberate cost trade-off documented in docs/agents/release-and-versioning.md — Linux is the cheap, fast feedback loop; Windows/macOS are reserved for post-merge.

Test plan

  • YAMLs parse
  • Build.cs compiles
  • CI green on this PR
  • Post-merge: confirm windows-latest + macos-latest fire when a commit lands on release/v11 (next hotfix or sentinel commit)

Refs milestone #13.

🤖 Generated with Claude Code

Closes #293. Parity follow-up to #291 (which extended PR-validation
triggers to release/v*). After this:

- windows-latest.yml — pull_request trigger unchanged (still PR-only on
  main, by design); push trigger extends to main AND release/v*.
- macos-latest.yml — same.
- build/Build.CI.GitHubActions.cs — OnPushBranches on both windows and
  macos [GitHubActions] entries gain ReleaseBranchPattern alongside
  MainBranch, so next auto-regen produces matching YAML.

Effect: a hotfix merged to release/v11 now triggers Windows + macOS
post-merge validation, same as merges to main. Cross-platform
regressions in release-branch hotfixes surface immediately as red
commits on the release branch, not later when someone tries to cut a
release.

Out of scope: extending pull_request triggers on windows/macos to fire
on release/v* PRs. The current PR gate (ubuntu-latest only) is a
deliberate cost trade-off documented in docs/agents/release-and-versioning.md.

Refs milestone #13.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ChrisonSimtian ChrisonSimtian merged commit 2e5ff68 into main May 29, 2026
1 check passed
@ChrisonSimtian ChrisonSimtian deleted the ci/extend-push-triggers-to-release-branches branch May 29, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend windows-latest + macos-latest post-merge triggers to release/v*

1 participant