Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set FlagNetStandard1XDependencies=true and fix issues #96795

Merged
merged 6 commits into from
Jan 25, 2024

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
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 10, 2024
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
191 of 194 checks passed
@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