Skip to content

Set FlagNetStandard1XDependencies=true and fix issues - #96795

Merged
akoeplinger merged 6 commits into
dotnet:mainfrom
akoeplinger:flag-ns1x-dependencies
Jan 25, 2024
Merged

Set FlagNetStandard1XDependencies=true and fix issues#96795
akoeplinger merged 6 commits into
dotnet:mainfrom
akoeplinger:flag-ns1x-dependencies

Conversation

@akoeplinger

@akoeplingerakoeplinger commented Jan 10, 2024

Copy link
Copy Markdown
Member

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.

@ghostghost 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
akoeplingerforce-pushed the flag-ns1x-dependencies branch from 31bcaf0 to 4b84e81CompareJanuary 22, 2024 15:06
Comment threadeng/Versions.props
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

@ViktorHoferViktorHofer left a comment

Copy link
Copy Markdown
Member

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:mainJan 25, 2024
@akoeplinger
akoeplinger deleted the flag-ns1x-dependencies branch January 25, 2024 14:19
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Feb 25, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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

@akoeplinger@jaredpar@ViktorHofer@eerhardt@ericstj