Skip to content

fix(release): drop ${{ }} braces from a workflow_call output description#19

Merged
bitwise-fast-forward-merge[bot] merged 1 commit into
bitwise-media-group:mainfrom
dmccaffery:fix/invalid-workflow
Jun 28, 2026
Merged

fix(release): drop ${{ }} braces from a workflow_call output description#19
bitwise-fast-forward-merge[bot] merged 1 commit into
bitwise-media-group:mainfrom
dmccaffery:fix/invalid-workflow

Conversation

@dmccaffery

Copy link
Copy Markdown
Collaborator

GitHub evaluates ${{ }} expressions inside a workflow_call output description. The release_created output description embedded a literal if: ${{ needs.release.outputs.release_created }} as caller guidance, but the needs context is out of scope there, so GitHub rejected release.yaml as an invalid workflow file. Every run that reaches it (self-release and any consumer caller) startup-failed with zero jobs from e0317e4 onward.

Drop the braces from the example (and the matching lead-in comment); a job-level if: needs none. Add a note so the braces do not creep back in.

GitHub evaluates ${{ }} expressions inside a workflow_call output
description. The release_created output description embedded a literal
`if: ${{ needs.release.outputs.release_created }}` as caller guidance, but
the `needs` context is out of scope there, so GitHub rejected release.yaml
as an invalid workflow file. Every run that reaches it (self-release and any
consumer caller) startup-failed with zero jobs from e0317e4 onward.

Drop the braces from the example (and the matching lead-in comment); a
job-level `if:` needs none. Add a note so the braces do not creep back in.

Signed-off-by: Deavon M. McCaffery <dmccaffery@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Note

Merging this PR: this repository merges by fast-forward so every
commit keeps its original signature. The GitHub merge button is not used.
Once this PR is approved and all checks pass, a maintainer merges it by
commenting /merge on the PR.

The branch must be up to date with main (rebased and re-signed) to
fast-forward. If /merge reports it is not fast-forwardable, rebase onto
main and comment /merge again.

@dmccaffery

Copy link
Copy Markdown
Collaborator Author

/auto-merge

@bitwise-fast-forward-merge bitwise-fast-forward-merge Bot added the auto-merge Fast-forward this PR once it is approved and all required checks pass label Jun 28, 2026
@bitwise-fast-forward-merge

Copy link
Copy Markdown
Contributor

Note

Auto-merge armed. Once this PR is approved and every required check
passes, it will be fast-forwarded into the base branch automatically,
preserving every commit signature.

Remove the auto-merge label to cancel. If the branch is not
fast-forwardable, rebase onto the base branch and re-sign — CI re-runs
and the merge retries.

@bitwise-fast-forward-merge

Copy link
Copy Markdown
Contributor

Fast-forwarded main to e148d2c0905c — original signature preserved, no re-sign.

@bitwise-fast-forward-merge bitwise-fast-forward-merge Bot merged commit e148d2c into bitwise-media-group:main Jun 28, 2026
11 checks passed
@dmccaffery dmccaffery deleted the fix/invalid-workflow branch June 28, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Fast-forward this PR once it is approved and all required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants