Skip to content

Conversation

@akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Jan 10, 2024

The only product change is upgrading Microsoft.DiaSymReader from 1.4.0 to 2.0.0 in ILCompiler.TypeSystem.csproj, the rest are test changes.

@ghost ghost assigned akoeplinger Jan 10, 2024
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 10, 2024
@akoeplinger akoeplinger force-pushed the flag-ns1x-dependencies branch from 31bcaf0 to 4b84e81 Compare January 22, 2024 15:06
ViktorHofer added a commit to ViktorHofer/roslyn that referenced this pull request Jan 24, 2024
ViktorHofer added a commit to ViktorHofer/roslyn-sdk that referenced this pull request Jan 24, 2024
Found in dotnet/runtime#96795 (comment)

These packages shouldn't be referenced on modern TFMs as the types are already provided inbox by the framework.
@akoeplinger akoeplinger marked this pull request as ready for review January 24, 2024 15:14
jaredpar pushed a commit to dotnet/roslyn that referenced this pull request Jan 24, 2024
* Don't reference S.R.C.Unsafe in net7.0 or newer

Found in dotnet/runtime#96795 (comment)

* Update src/Compilers/Core/Portable/Microsoft.CodeAnalysis.csproj
Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

This is great 💯

@akoeplinger akoeplinger merged commit 754a554 into dotnet:main Jan 25, 2024
@akoeplinger akoeplinger deleted the flag-ns1x-dependencies branch January 25, 2024 14:19
@github-actions github-actions bot locked and limited conversation to collaborators Feb 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants