Skip to content

Microsoft.DotNet.SignCheckLibrary: Update Newtonsoft.Json reference to #9962

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

Closed
wants to merge 1 commit into from

Conversation

radical
Copy link
Member

@radical radical commented Jul 12, 2022

.. 13.0.1 to match rest of the repo. Also, 9.0.1 triggers component
governance warnings.

To double check:

cc @lewing @akoeplinger @am11

.. 13.0.1 to match rest of the repo. Also, 9.0.1 triggers component
governance warnings.
@garath
Copy link
Member

garath commented Jul 12, 2022

@ChadNedzlek FYI.

This PR overlaps with #9885

@ChadNedzlek
Copy link
Contributor

Also this one: #9777. Apparently we are trying to solve this 3 different ways all at once.

@dotnet dotnet deleted a comment from azure-pipelines bot Jul 12, 2022
@am11
Copy link
Member

am11 commented Jul 13, 2022

LGTM, whichever approach we end up using, it will likely unblock dotnet/runtime#71180.

@ChadNedzlek
Copy link
Contributor

If I can get the build to build, I'll try and merge this. :-)

@MihaZupan
Copy link
Member

@ChadNedzlek should this be merged?

@garath
Copy link
Member

garath commented Aug 8, 2022

should this be merged?

This reference was updated as part of #10126, so this PR is no longer needed.

@garath garath closed this Aug 8, 2022
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.

5 participants