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 to version 6.0 and net60 #161

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marner2
Copy link
Contributor

@marner2 marner2 commented Oct 29, 2022

Updated to major version 6 to mirror the .net version.

This all seems to "just work."

@panmanphil
Copy link
Collaborator

I am inclined to say that everything looks fine here, and thank you for the work! I'm not currently working with .net so my memory is fuzzy on one aspect of this. Since this is a library, we want to support as many runtimes as possible. Reading back through earlier commits we said that after v 5 of the runtime there is no longer any need do anything to support .net core and .net framework, that anything built with v 5 or later will support dotnetstandard 2.1. Is that goal met now, there is nothing left for us to do?

@marner2
Copy link
Contributor Author

marner2 commented Oct 30, 2022

Looking at it now, we should probably do version ranges for Microsoft.AspNetCore.Mvc.NewtonsoftJson so we don't force people to use binding redirects if another version comes out

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.

None yet

2 participants