-
Notifications
You must be signed in to change notification settings - Fork 4k
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 referenced MSBuild to 17.3.4 #73113
Conversation
Might need another update somewhere.
|
To account for newer binlog versions emitted by build tools.
Tried to update to latest which wasn't on our feeds, so I tried to push it to the feeds but failed, so I backed it down to the latest in the feeds, which should be new enough. |
The CI test failures are expected and not fixable. |
@marcpopMSFT are you good on the .NET SDK to absorb this in 8.0.1xx if we merge here? |
Yes, the 8.0.1xx sdk should take this codeflow for a June release. |
Matches dotnet/roslyn#73113 Add a reference to NuGet.Frameworks (but don't bundle our own version) to the generator and the EnC tests.
[release/8.0] Update dependencies from dotnet/roslyn - Bump MSBuild dependency to 17.3.4 Matches dotnet/roslyn#73113 Add a reference to NuGet.Frameworks (but don't bundle our own version) to the generator and the EnC tests. - add correct feed for MSBuild packages
Fixes #72812
@marcpopMSFT @jaredpar @rainersigwald