Skip to content

SCM - fix rendering issue with incoming changes node#276055

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/fascinating-dormouse
Nov 7, 2025
Merged

SCM - fix rendering issue with incoming changes node#276055
lszomoru merged 1 commit intomainfrom
lszomoru/fascinating-dormouse

Conversation

@lszomoru
Copy link
Member

@lszomoru lszomoru commented Nov 7, 2025

Related #275820

Copilot AI review requested due to automatic review settings November 7, 2025 12:05
@lszomoru lszomoru enabled auto-merge (squash) November 7, 2025 12:05
@lszomoru lszomoru self-assigned this Nov 7, 2025
@lszomoru lszomoru added the scm General SCM compound issues label Nov 7, 2025
@lszomoru lszomoru added this to the November 2025 milestone Nov 7, 2025
@lszomoru lszomoru force-pushed the lszomoru/fascinating-dormouse branch from 3b68f18 to 6ae01a1 Compare November 7, 2025 12:08
Copy link
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

This PR adds a conditional check to only show incoming changes in the SCM history graph when the remote history item reference is actually part of the graph being displayed.

  • Changed behavior: The incoming changes node is now only added if both the configuration is enabled AND the remote reference exists in the current graph's history item references

@lszomoru lszomoru merged commit 88bd915 into main Nov 7, 2025
28 checks passed
@lszomoru lszomoru deleted the lszomoru/fascinating-dormouse branch November 7, 2025 12:53
lszomoru added a commit that referenced this pull request Nov 7, 2025
SCM - fix rendering issue with incoming/outgoing changes nodes
lszomoru added a commit that referenced this pull request Nov 7, 2025
SCM - fix rendering issue with incoming/outgoing changes nodes
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Dec 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

scm General SCM compound issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants