-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Revert "Revert "Remove downlevel runtimes from global.json"" #40701
Conversation
Should work now that we're explicitly restoring 3.1 for this step |
Test build worked |
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.
Approving but I prefer the "rollForward": "major"
approach
May not be possible for local tools - dotnet/sdk#10375. @marcpopMSFT @dsplaisted is this supported now? |
I think dotnet/sdk#10375 was about a command-line option to enable tool roll-forward. We never did that. However, a tool should be able to specify "rollForward": "major" in its runtimeconfig.json file and that should work. |
I'm just going to take this as-is then since we're downloading & running the tool in the script |
Reverts #40699 & adds an explicit 3.1 restore before trying to use
BinlogToSln