-
Notifications
You must be signed in to change notification settings - Fork 228
[main] Source code updates from dotnet/sdk #3754
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
[main] Source code updates from dotnet/sdk #3754
Conversation
|
Caution 🛑 Conflict detectedA conflict was detected when trying to update this PR with changes from https://github.com/dotnet/sdk/tree/e300dea82e0248b0801f706914c96073c1bd5350. The conflicts in the following files need to be manually resolved so that automated codeflow can resume for this PR:
ℹ️ To resolve the conflict, please follow these steps:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Diff: https://github.com/dotnet/sdk/compare/e300dea82e0248b0801f706914c96073c1bd5350..e300dea82e0248b0801f706914c96073c1bd5350 From: dotnet/sdk@e300dea To: dotnet/sdk@e300dea The following files had conflicts that were resolved by a user: - src/sdk/eng/Signing.props - src/sdk/eng/Versions.props - src/sdk/src/Layout/redist/targets/BundledTemplates.targets - src/sdk/src/Layout/redist/targets/GeneratePackagePruneData.targets
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
|
@marcpopMSFT in case you haven't noticed this PR. This brings in all the remaining sdk changes since November 3rd. |
Missed version update in dotnet/source-build-reference-packages#1482
|
@dotnet/source-build do we need to temporarily disable the dotnet watch test?
|
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
@javiercn - can you please take a look at this failure? |
|
@dotnet/fsharp for |
|
Note PRs from original repository included in this codeflow update: 💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Is this PR the first ever scenario run on SDK=11 ? |
|
Yes. This is the PR that brings in the branding updates to 11.0.100 in SDK. |
|
OK, that might explain it. F# historically depends on a private (= was wrong for F# to do it) target from nuget.client's Nuget.Build.Tasks.Pack.targets which might be optimized away (for good reasons) depending on the scenario. Did a change to verify it here in this PR |
|
For the record on the NuGet issues in scenario tests:
|
|
Hmm, the It's copying to As a rule of thumb we should be harvesting only the 11 templates and not the 10 templates. We used to do some of the downlevel templates back in 8 or 9, but we stopped doing that. |
Note
This is a codeflow update. It may contain both source code changes from
the source repo
as well as dependency updates. Learn more here.
This pull request brings the following source code changes
From https://github.com/dotnet/sdk
Diff the source with this PR branch