Skip to content

[main] Source code updates from dotnet/dotnet#81958

Merged
dotnet-maestro[bot] merged 2 commits intomainfrom
darc-main-51dc08c0-4113-4a50-9f96-16a4ec9fd9d8
Jan 12, 2026
Merged

[main] Source code updates from dotnet/dotnet#81958
dotnet-maestro[bot] merged 2 commits intomainfrom
darc-main-51dc08c0-4113-4a50-9f96-16a4ec9fd9d8

Conversation

@dotnet-maestro
Copy link
Contributor

Note

This is a codeflow update. It may contain both source code changes from
the VMR
as well as dependency updates. Learn more here.

This pull request brings the following source code changes

From https://github.com/dotnet/dotnet

Updated Dependencies

Associated changes in source repos

Diff the source with this PR branch
darc vmr diff --name-only https://github.com/dotnet/dotnet:db3b7424251b47d9474dbbe3b4fa7a81fe2915b7..https://github.com/dotnet/roslyn:darc-main-51dc08c0-4113-4a50-9f96-16a4ec9fd9d8

Copy link
Contributor

@dotnet-policy-service dotnet-policy-service bot left a comment

Choose a reason for hiding this comment

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

Auto-approve

</ItemGroup>
<ItemGroup Condition="'$(DotNetBuildSourceOnly)' != 'true'">
<PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" PrivateAssets="all" />
<PackageReference Include="Roslyn.Diagnostics.Analyzers" PrivateAssets="all" />
Copy link
Member

Choose a reason for hiding this comment

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

Doesn't this break the VMR?

Copy link
Member

Choose a reason for hiding this comment

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

@dotnet/product-construction can we investigate why this attempted to revert this change? The corresponding locations in the VMR's commit have not been modified in 2 years. I'm extremely concerned that if this is getting overwritten, other fixes and changes may be overridden by the code flow.

Copy link
Member

Choose a reason for hiding this comment

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

Jan moved Roslyn.Diagnostics.Analyzers out of this item group, to the unconditioned one, in #81856 but that broke the last VMR flow PR from Roslyn and so he pushed a revert commit to that PR: dotnet/dotnet#4084

Copy link
Member

Choose a reason for hiding this comment

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

Ah, so this is a rare instance of git blame kinda lying to me. Thanks.

@333fred 333fred dismissed their stale review January 12, 2026 06:11

Incorrect

@dotnet-maestro dotnet-maestro bot merged commit 13d3fe0 into main Jan 12, 2026
36 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-main-51dc08c0-4113-4a50-9f96-16a4ec9fd9d8 branch January 12, 2026 06:16
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Jan 12, 2026
@davidwengier davidwengier modified the milestones: Next, 18.4 Jan 27, 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.

2 participants