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

Update dependency versions for 4.2.0 publishing #61265

Merged
merged 7 commits into from
May 13, 2022

Conversation

333fred
Copy link
Member

@333fred 333fred commented May 12, 2022

Updates our nuget versions for 4.2.0, so that all published binaries will depend on publicly-available non-prerelease packages. As part of this, I had to break add multiple versions of MSBuild, as Microsoft.VisualStudio.Shell.15.0 is now advertising their dependency on MSBuild 17.1.0. There was always an indirect dependency there, but it was not included in their nuspec.

Updates our nuget versions for 4.2.0, so that all published binaries will depend on publicly-available non-prerelease packages. As part of this, I had to break add multiple versions of MSBuild, as Microsoft.VisualStudio.Shell.15.0 is now advertising their dependency on MSBuild 17.1.0. There was always an indirect dependency there, but it was not included in their nuspec.
eng/Versions.props Outdated Show resolved Hide resolved
sharwell
sharwell previously approved these changes May 12, 2022
Copy link
Member

@jaredpar jaredpar left a comment

Choose a reason for hiding this comment

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

netcoreapp3.1 is an out of support runtime. We should not be complicating our build to support it. Literally every other .NET team is removing this TF. We should not be the outlier because it will long term result in us hacking our build over and over again.

@333fred 333fred requested a review from a team as a code owner May 12, 2022 18:23
@@ -5,7 +5,7 @@
<OutputType>Library</OutputType>
<RootNamespace>Microsoft.CodeAnalysis</RootNamespace>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<TargetFrameworks>netcoreapp3.1;net472</TargetFrameworks>
Copy link
Member Author

@333fred 333fred May 12, 2022

Choose a reason for hiding this comment

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

Source changes in this project are to deal with newly annotated APIs.

@sharwell sharwell dismissed their stale review May 12, 2022 18:47

Withdrawing approval due to removal of a previously supported TFM, which has always been extremely frustrating to me as a user

@genlu
Copy link
Member

genlu commented May 12, 2022

netcoreapp3.1 is an out of support runtime.

Are we considering the move from 3.1 to .Net 6 in main?

eng/Versions.props Outdated Show resolved Hide resolved
@333fred 333fred enabled auto-merge May 12, 2022 22:23
@JoeRobich
Copy link
Member

/azp run roslyn-integration-CI

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@333fred 333fred merged commit 46c8f4f into dotnet:release/dev17.2 May 13, 2022
@333fred 333fred deleted the update-versions branch May 13, 2022 16:47
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.

5 participants