-
Notifications
You must be signed in to change notification settings - Fork 24
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
Tooling feed updates to support .NET 9 #549
Conversation
Few things have been identified to come up in the follow-up PR -
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes seem good overall - my only question is if net5-isolated needs to be removed at this time. I'd probably rather see that be a separate set of changes if we're going to be doing that, unless it is required as part of getting net9 out.
Stale review - changes have since come in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing the positioning back to "request changes" to ensure .NET 5 is not removed with this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @mattchenderson 's comment about .NET 5 here.
It's something we should be doing, but needs coordination and shouldn't be combined in this PR
resolves #550
Tooling feed updates to support .NET 9
Diff between 4.84.0 and 4.85.0 - https://gist.github.com/surgupta-msft/737c4f7048bc5df898ca0bfaa5ee253c/revisions
Diff between net8 and net9 - https://gist.github.com/surgupta-msft/0f978c67ee04aa2cf0cc9dfd0b27a851/revisions