Skip to content

ci: fix notify.yaml startup failure from empty expression - #18

Merged
Synertry merged 1 commit into
mainfrom
ci/notify-empty-expression
Jun 13, 2026
Merged

ci: fix notify.yaml startup failure from empty expression#18
Synertry merged 1 commit into
mainfrom
ci/notify-empty-expression

Conversation

@Synertry

Copy link
Copy Markdown
Owner

Summary

  • An empty ${{ }} written inside a run: block comment was parsed as an invalid GitHub Actions expression, so notify.yaml never compiled. It surfaced as a startup-failure run on every push (27 so far) and stopped the workflow from registering its release: published trigger (zero release runs ever). Reworded the comment to remove the literal. actionlint now passes clean.

An empty GitHub expression written literally in a run-block comment is parsed as an invalid expression, so notify.yaml never compiled. That surfaced as a startup-failure run on every push and prevented the release trigger from registering. Reworded the comment to drop the literal.

Signed-off-by: Synertry <60792595+Synertry@users.noreply.github.com>
@github-actions github-actions Bot added area/CI GitHub workflows, Dependabot, labelers, scripts type/ci Changes to CI configuration files and scripts size/xs Extra small diff (< 25 lines, 1 file) status/pr-test-passed Lightweight PR test workflow passed (granted by pr-test.yaml) labels Jun 13, 2026
@Synertry
Synertry merged commit cb327b2 into main Jun 13, 2026
7 checks passed
@github-actions
github-actions Bot deleted the ci/notify-empty-expression branch June 13, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI GitHub workflows, Dependabot, labelers, scripts size/xs Extra small diff (< 25 lines, 1 file) status/pr-test-passed Lightweight PR test workflow passed (granted by pr-test.yaml) type/ci Changes to CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant