-
Notifications
You must be signed in to change notification settings - Fork 446
Conversation
build/sources.props
Outdated
@@ -7,6 +7,8 @@ | |||
$(RestoreSources); | |||
https://dotnet.myget.org/F/aspnetcore-dev/api/v3/index.json; | |||
https://dotnet.myget.org/F/aspnetcore-tools/api/v3/index.json; | |||
https://dotnet.myget.org/F/dotnet-core/api/v3/index.json; |
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.
@natemcmaster FYI in case you have an alternative.
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.
It's temporary
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 is fine for local builds and temporary workarounds, but would break in Universe if you don't also update that to pull the new corefxlab packages.
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.
LGTM from a product-code perspective. Need to rebase and resolve conflicts. Also, deal with the package sources as per Nate's comment.
21aab4b
to
0ef8ba9
Compare
No description provided.