Skip to content

include source repo commit links in backflow commit message#6213

Merged
adamzip merged 1 commit intodotnet:mainfrom
adamzip:include-diffs-in-backflow-commit-msg
Apr 23, 2026
Merged

include source repo commit links in backflow commit message#6213
adamzip merged 1 commit intodotnet:mainfrom
adamzip:include-diffs-in-backflow-commit-msg

Conversation

@adamzip
Copy link
Copy Markdown
Contributor

@adamzip adamzip commented Apr 23, 2026

#5317

Add the last SHA and currently flowing SHA from the source repo, with their diff. Just like in forward flows.

Repro: maestro-auth-test/sdk@0b8620b

Copilot AI review requested due to automatic review settings April 23, 2026 14:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates DARC VMR backflow behavior to include VMR source commit links (previous/current SHAs and a compare link) in the backflow commit message, aligning it with the link-rich messaging used in forward flows.

Changes:

  • Pass LastFlows through backflow patch/commit plumbing so the commit message can reference the prior flown VMR SHA.
  • Extend the backflow commit message with Diff, From, and To links for the VMR commit range being backflowed.

Comment thread src/Microsoft.DotNet.Darc/DarcLib/VirtualMonoRepo/VmrBackflower.cs
Comment thread src/Microsoft.DotNet.Darc/DarcLib/VirtualMonoRepo/VmrBackflower.cs
@adamzip adamzip requested review from dkurepa and premun April 23, 2026 14:13
@adamzip adamzip merged commit 36ae706 into dotnet:main Apr 23, 2026
11 of 13 checks passed
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