Skip to content

ci: Reject placeholder internal-only commit messages - #131

Merged
copybara-service[bot] merged 1 commit into
masterfrom
test_1017044682
Nov 9, 2026
Merged

copybara-service[bot] merged 1 commit into
masterfrom
test_1017044682

Conversation

@copybara-service

Copy link
Copy Markdown

ci: Reject placeholder internal-only commit messages

Copybara exports from Google's internal repository require a public
commit description, or fall back to a placeholder when none is provided.
Authors also sometimes write a boilerplate placeholder when they expect
a change not to be exported, and if the change touches an exported file
anyway, that placeholder lands in the git history.

Fail the pre-commit workflow if the commit message contains one of
those placeholder phrases. Fetch depth 2 in actions/checkout so that
pull_request runs can inspect pull_request.head.sha rather than
GitHub's synthetic merge commit (Merge <sha> into <sha>), while still
running pre-commit on the merged tree.

@copybara-service
copybara-service Bot force-pushed the test_1017044682 branch 3 times, most recently from d448245 to a24ac75 Compare November 9, 2026 13:44
Copybara exports from Google's internal repository require a public
commit description, or fall back to a placeholder when none is provided.
Authors also sometimes write a boilerplate placeholder when they expect
a change not to be exported, and if the change touches an exported file
anyway, that placeholder lands in the git history.

Fail the `pre-commit` workflow if the commit message contains one of
those placeholder phrases. Fetch depth 2 in `actions/checkout` so that
`pull_request` runs can inspect `pull_request.head.sha` rather than
GitHub's synthetic merge commit (`Merge <sha> into <sha>`), while still
running `pre-commit` on the merged tree.

PiperOrigin-RevId: 1017279458
@copybara-service
copybara-service Bot merged commit a51ab8a into master Nov 9, 2026
@copybara-service
copybara-service Bot deleted the test_1017044682 branch November 9, 2026 13:53
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