Skip to content

feat(tpl-terragrunt): add plan-gate job as stable required status check - #17

Merged
sergioaten merged 1 commit into
mainfrom
fix/pr-number-from-commit-msg
Mar 19, 2026
Merged

sergioaten merged 1 commit into
mainfrom
fix/pr-number-from-commit-msg

Conversation

@sergioaten

Copy link
Copy Markdown
Contributor

The matrix plan job reports as "plan / plan ({stack}, {slug})" which is stack-name-dependent and can't be used directly as a branch protection required check. The comment job always passes regardless of plan outcome.

plan-gate is a non-matrix job that:

  • always runs when action=plan (even with no iac changes)
  • fails if any plan matrix job failed
  • passes if plan succeeded or was skipped (no iac changes)

Required status check in branch protection should be "plan / plan-gate".

@sergioaten
sergioaten merged commit 80110ac into main Mar 19, 2026
@sergioaten
sergioaten deleted the fix/pr-number-from-commit-msg branch March 19, 2026 19:04
The matrix plan job reports as "plan / plan ({stack}, {slug})" which is
stack-name-dependent and can't be used directly as a branch protection
required check. The comment job always passes regardless of plan outcome.

plan-gate is a non-matrix job that:
- always runs when action=plan (even with no iac changes)
- fails if any plan matrix job failed
- passes if plan succeeded or was skipped (no iac changes)

Required status check in branch protection should be "plan / plan-gate".

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sergioaten
sergioaten restored the fix/pr-number-from-commit-msg branch March 19, 2026 22:09
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.

1 participant