Skip to content

Fix publish job dry_run handling for schedule trigger#20

Merged
blink1073 merged 3 commits into
Calysto:mainfrom
blink1073:fix-changelog-again
Mar 24, 2026
Merged

Fix publish job dry_run handling for schedule trigger#20
blink1073 merged 3 commits into
Calysto:mainfrom
blink1073:fix-changelog-again

Conversation

@blink1073
Copy link
Copy Markdown
Contributor

Summary

  • publish job used inputs.dry_run directly, which is empty/undefined when triggered by schedule
  • Apply the same ternary pattern already used by the release job: github.event_name == 'schedule' && 'true' || inputs.dry_run

Test plan

  • Verify scheduled runs set skip-existing and verbose to true in the publish step
  • Verify workflow_dispatch runs still respect the dry_run input

@blink1073 blink1073 enabled auto-merge (squash) March 24, 2026 12:26
@blink1073 blink1073 disabled auto-merge March 24, 2026 12:26
@blink1073 blink1073 enabled auto-merge (squash) March 24, 2026 12:26
@blink1073 blink1073 merged commit fb361e8 into Calysto:main Mar 24, 2026
18 checks passed
@blink1073 blink1073 deleted the fix-changelog-again branch March 24, 2026 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant