Skip to content

chore(ci): remove squash requirement - #333892

Draft
Raymond Zhao (rzhao271) wants to merge 1 commit into
mainfrom
rzhao271/mad-sailfish
Draft

chore(ci): remove squash requirement#333892
Raymond Zhao (rzhao271) wants to merge 1 commit into
mainfrom
rzhao271/mad-sailfish

Conversation

@rzhao271

Copy link
Copy Markdown
Collaborator
  • Tested on ADO

Copilot AI balanced review requested due to automatic review settings September 1, 2026 23:06
@rzhao271 Raymond Zhao (rzhao271) added this to the 1.137.0 milestone Sep 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Manually queued builds now unintentionally bypass trailer validation.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Removes the squash requirement while retaining MSRC trailer validation.

Changes:

  • Supports multi-commit PRs.
  • Requires at least one valid Msrc-Case-Id trailer.
  • Skips validation for non-PR builds.
File summaries
File Description
build/azure-pipelines/product-build-ado-ci.yml Updates commit-trailer validation logic.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +263 to +267
# Only validate trailers for PR builds
if [ "$(Build.Reason)" != "PullRequest" ]; then
echo "Skipping trailer check for non-PR builds"
exit 0
fi
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.

2 participants