Skip to content

DEVEX-1628: optional skip_ci on php-common-bump#130

Open
pdodgen-revparts wants to merge 5 commits into
mainfrom
DEVEX-1628-common-bump-build-on-nightly
Open

DEVEX-1628: optional skip_ci on php-common-bump#130
pdodgen-revparts wants to merge 5 commits into
mainfrom
DEVEX-1628-common-bump-build-on-nightly

Conversation

@pdodgen-revparts

Copy link
Copy Markdown
Contributor

Adds an optional skip_ci input (default true = current behavior) so nightly common bumps can omit [skip actions] and trigger a build/tag. Backward-compatible: no existing caller changes behavior. Nightly workflows opt in via separate PRs.

Refs DEVEX-1628

@cursor

cursor Bot commented Jun 10, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes shared CI commit behavior used by many repos; default true limits blast radius, but skip_ci: false will trigger push pipelines and integration deploys when nightly workflows opt in.

Overview
Adds an optional skip_ci input (default true) to the shared php-common-bump reusable workflow so bump commits only append [skip actions] when callers want to suppress CI. The commit step now reads skip_ci and commit_message via env and builds the message suffix conditionally instead of always skipping workflows.

This unblocks DEVEX-1628: nightly common-update jobs can opt in with skip_ci: false in follow-up PRs so revolutionparts/common bumps on main trigger build/tag (and integration deploy where applicable). Existing callers (merge-path upgrade-common, daemons, batch, etc.) keep today’s behavior without changes.

Also adds design and implementation docs under docs/superpowers/ describing the nightly opt-in rollout across 11 service repos.

Reviewed by Cursor Bugbot for commit 4dc932c. Bugbot is set up for automated code reviews on this repo. Configure here.

@pdodgen-revparts pdodgen-revparts marked this pull request as ready for review June 10, 2026 20:58
@pdodgen-revparts pdodgen-revparts requested a review from a team as a code owner June 10, 2026 20: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.

1 participant