Skip to content

Upgrade newtonsoft dependency to 13.0.1#8059

Merged
mmitche merged 2 commits intodotnet:release/6.0from
MichaelSimons:upgrade-newtonsoft
Oct 19, 2021
Merged

Upgrade newtonsoft dependency to 13.0.1#8059
mmitche merged 2 commits intodotnet:release/6.0from
MichaelSimons:upgrade-newtonsoft

Conversation

@MichaelSimons
Copy link
Member

@MichaelSimons MichaelSimons commented Oct 18, 2021

This is needed in source-build as a result of NuGet updating to 13.0.1 (NuGet/NuGet.Client#4167)

@MichaelSimons MichaelSimons marked this pull request as ready for review October 18, 2021 22:01
@MichaelSimons
Copy link
Member Author

@MattGal - can you tell me the process for getting this merged in release/6.0?

@MattGal
Copy link
Member

MattGal commented Oct 18, 2021

@MattGal - can you tell me the process for getting this merged in release/6.0?

Perhaps there should be one, but AFAIK it's just "check in to main first, cherry pick and resolve conflicts to needed release branches, release branches <= 5.0 require leadership team approval".

@markwilkie
Copy link
Member

Policy for servicing is here: https://github.com/dotnet/arcade/blob/main/Documentation/Policy/ArcadeServicing.md

In general though, backports should get an explicit approval from someone in engineering - which already happened. :) And yes, the idea is to use main as the baking mechanism.

Also, cc'ing @mmitche for visibility.

@mmitche
Copy link
Member

mmitche commented Oct 19, 2021

If this is required for 6.0, we really don't have time for baking. Merging

@mmitche mmitche merged commit 85f3aa1 into dotnet:release/6.0 Oct 19, 2021
@MichaelSimons MichaelSimons deleted the upgrade-newtonsoft branch October 19, 2021 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants