-
Notifications
You must be signed in to change notification settings - Fork 65
Switch to new Microsoft.DotNet.SharedFramework.Sdk #909
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
Conversation
…its shared framework and installers.
Signed-off-by: Jeremy Koritzinsky <jekoritz@microsoft.com>
Signed-off-by: Jeremy Koritzinsky <jekoritz@microsoft.com>
|
@jkoritzinsky Can you resolve the conflicts and figure out what still needs to be done to get WD onto the new SDK? |
|
All that's left is to resolve the conflicts (which are all from the Maestro auto-update PRs IIRC). Everything is working in this PR on the new SDK. I'll resolve the conflicts when I'm back from vacation. |
RussKie
left a comment
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.
Few questions
pkg/Directory.Build.props
Outdated
| <!-- | ||
| SDK currently does not know about net6.0 TargetFramework. | ||
| We need to manually add these to correctly resolve the targeting packs. | ||
| https://github.com/dotnet/sdk/issues/13950 |
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.
This appears to have been resolved.
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.
To remove these items, we need to update the global.json to reference a 6.0-based SDK.
…hey build on top of the standard SDKs. Signed-off-by: Jeremy Koritzinsky <jekoritz@microsoft.com>
|
I've addressed all of the feedback and CI is green again. @RussKie can you take another look? |
RussKie
left a comment
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.
![]()
Switch to the new unified tooling in the Microsoft.DotNet.SharedFramework.Sdk package.