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

[vs17.14] Revert change in _SplitProjectReferencesByFileExistence - #12035

#12035
Merged
rainersigwald merged 2 commits into
dotnet:vs17.14dotnet/msbuild:vs17.14from
YuliiaKovalova:dev/ykovalova/revert_SplitProjectReferencesByFileExistence_changesYuliiaKovalova/msbuild:dev/ykovalova/revert_SplitProjectReferencesByFileExistence_changesCopy head branch name to clipboard
Jun 19, 2025
Merged

[vs17.14] Revert change in _SplitProjectReferencesByFileExistence#12035
rainersigwald merged 2 commits into
dotnet:vs17.14dotnet/msbuild:vs17.14from
YuliiaKovalova:dev/ykovalova/revert_SplitProjectReferencesByFileExistence_changesYuliiaKovalova/msbuild:dev/ykovalova/revert_SplitProjectReferencesByFileExistence_changesCopy head branch name to clipboard

Conversation

@YuliiaKovalova

@YuliiaKovalova YuliiaKovalova commented Jun 17, 2025

Copy link
Copy Markdown
Member

Summary

Regression caused by change #11167 in _SplitProjectReferencesByFileExistence target. The decision was made to prioritize customer stability over the technical improvement, as the widespread impact couldn't be quickly addressed by affected users.

Customer Impact

Customers experienced breaking changes in project reference handling that require modifications on their end to resolve. The impact is widespread and cannot be easily mitigated by users without targets changes on their side (e.g a custom target like described here #11923).

Regression?

Yes, introduced by change #11167.

Risk

Very Low - Reverting to previous historical behavior that was proven stable.

Changes Made

Reverted the _SplitProjectReferencesByFileExistence target changes to restore the previous historical behavior.

Copilot AI review requested due to automatic review settings June 17, 2025 16:02
@YuliiaKovalova
YuliiaKovalova requested a review from a team as a code owner June 17, 2025 16:02

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 reverts the AssignProjectConfiguration dependency removal for the _SplitProjectReferencesByFileExistence target to restore historical behavior and avoid customer regressions, and bumps the version prefix for a new release.

  • Re-adds DependsOnTargets="AssignProjectConfiguration" attribute to the target declaration
  • Updates <VersionPrefix> from 17.14.13 to 17.14.14 for the upcoming release

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Tasks/Microsoft.Common.CurrentVersion.targets Restores DependsOnTargets="AssignProjectConfiguration" on the _SplitProjectReferencesByFileExistence target
eng/Versions.props Increments <VersionPrefix> to 17.14.14

@YuliiaKovalova YuliiaKovalova changed the title Revert change in _SplitProjectReferencesByFileExistence [vs17.14] Revert change in _SplitProjectReferencesByFileExistence Jun 17, 2025
@YuliiaKovalova YuliiaKovalova self-assigned this Jun 18, 2025
@rainersigwald
rainersigwald merged commit a129329 into dotnet:vs17.14 Jun 19, 2025
10 checks passed
This was referenced Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

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