Skip to content

PPNA-5781 Don't run version-update on skipped workflows (commits with "[skip ci]" etc)#185

Draft
mrfijas wants to merge 1 commit into
masterfrom
guard-version-update-against-skipped-CIs
Draft

PPNA-5781 Don't run version-update on skipped workflows (commits with "[skip ci]" etc)#185
mrfijas wants to merge 1 commit into
masterfrom
guard-version-update-against-skipped-CIs

Conversation

@mrfijas

@mrfijas mrfijas commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

WiP

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the version-update/trigger composite action so it does not dispatch version-update workflows when the source commit indicates CI should be skipped (e.g., [skip ci]), including when invoked from a workflow_run-based workflow.

Changes:

  • Derive source_sha/source_ref for both normal events and workflow_run, and surface them via outputs.
  • Detect GitHub Actions skip directives in the source commit message and skip dispatching when present (with a skip_reason output and updated summaries).
  • Document the new skip behavior in the README.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
version-update/trigger/action.yaml Adds commit-message skip detection and workflow_run source SHA/ref handling; enriches outputs and summary text.
README.md Documents skip-directive behavior for dev commits and tags, including workflow_run usage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread version-update/trigger/action.yaml Outdated
@mrfijas

mrfijas commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

cursor review

@jzyinq

jzyinq commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@cursor review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread version-update/trigger/action.yaml Outdated
@mrfijas mrfijas force-pushed the guard-version-update-against-skipped-CIs branch from 7d627e7 to 5d7d105 Compare June 26, 2026 11:58
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.

3 participants