Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update fromBranch message for clarity on availability - #6481

#6481
Merged
akoeplinger merged 3 commits into
maindotnet/arcade-services:mainfrom
akoeplinger-patch-1dotnet/arcade-services:akoeplinger-patch-1Copy head branch name to clipboard
Jul 7, 2026
Merged

Update fromBranch message for clarity on availability#6481
akoeplinger merged 3 commits into
maindotnet/arcade-services:mainfrom
akoeplinger-patch-1dotnet/arcade-services:akoeplinger-patch-1Copy head branch name to clipboard

Conversation

@akoeplinger

@akoeplinger akoeplinger commented Jul 7, 2026

Copy link
Copy Markdown
Member

Right now you just get "... being divergent from the previously flown branch" which is misleading.

Also fix the double-escaping when the branch name is available, we already wrap the string in backticks in the PR message.

Related issue: #6390

Right now you just get "... being divergent from the previously flown branch" which is misleading.

Also fix the double-escaping when the branch name is available, we already wrap the string in backticks in the PR message.

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.

Pull request overview

This PR updates the unsafe codeflow PR header text generation in PCS dependency flow to avoid misleading wording when the previously-flown branch isn’t known, and to prevent double-wrapping the branch value in Markdown backticks.

Changes:

  • Stop pre-wrapping fromBranch in backticks to avoid double-escaping in the rendered PR message.
  • Replace the generic fallback "branch" with an explicit placeholder indicating the branch name is unavailable.

akoeplinger and others added 2 commits July 7, 2026 16:43
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@akoeplinger
akoeplinger enabled auto-merge July 7, 2026 16:59
@akoeplinger
akoeplinger merged commit 5b2a66f into main Jul 7, 2026
8 of 13 checks passed
@akoeplinger
akoeplinger deleted the akoeplinger-patch-1 branch July 7, 2026 17:51
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.