Skip to content

Include PR number in automerge commit subject#91

Merged
mattleibow merged 1 commit into
mainfrom
dev/fix-automerge-commit-message
May 15, 2026
Merged

Include PR number in automerge commit subject#91
mattleibow merged 1 commit into
mainfrom
dev/fix-automerge-commit-message

Conversation

@mattleibow

Copy link
Copy Markdown
Collaborator

The squash merge subject was hardcoded as Update API docs from latest CI build without appending (#N), making it impossible to trace commits back to their source PR in git log.

This adds (#$PR_NUMBER) to the --subject flag so commits look like:

Update API docs from latest CI build (#91)

The squash merge subject was hardcoded without the PR number,
making it hard to trace commits back to their source PR in
git log.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit e395709:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/automerge-docs.yml ✅Succeeded

For more details, please refer to the build report.

@mattleibow mattleibow merged commit 36bf4fe into main May 15, 2026
3 checks passed
@mattleibow mattleibow deleted the dev/fix-automerge-commit-message branch May 15, 2026 19:59
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