Skip to content
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

[automated] Merge branch 'vs17.9' => 'main' #9559

Merged
merged 14 commits into from
Dec 21, 2023

Conversation

dotnet-maestro-bot
Copy link
Contributor

I detected changes in the vs17.9 branch which have not been merged yet to main. I'm a robot and am configured to help you automatically keep main up to date, so I've opened this PR.

This PR merges commits made on vs17.9 by the following committers:

Instructions for merging from UI

This PR will not be auto-merged. When pull request checks pass, complete this PR by creating a merge commit, not a squash or rebase commit.

merge button instructions

If this repo does not allow creating merge commits from the GitHub UI, use command line instructions.

Instructions for merging via command line

Run these commands to merge this pull request from the command line.

git fetch
git checkout vs17.9
git pull --ff-only
git checkout main
git pull --ff-only
git merge --no-ff vs17.9

# If there are merge conflicts, resolve them and then run git merge --continue to complete the merge
# Pushing the changes to the PR branch will re-trigger PR validation.
git push https://github.com/dotnet-maestro-bot/msbuild HEAD:merge/vs17.9-to-main
or if you are using SSH
git push git@github.com:dotnet-maestro-bot/msbuild HEAD:merge/vs17.9-to-main

After PR checks are complete push the branch

git push

Instructions for resolving conflicts

⚠️ If there are merge conflicts, you will need to resolve them manually before merging. You can do this using GitHub or using the command line.

Instructions for updating this pull request

Contributors to this repo have permission update this pull request by pushing to the branch 'merge/vs17.9-to-main'. This can be done to resolve conflicts or make other changes to this pull request before it is merged.

git checkout -b merge/vs17.9-to-main main
git pull https://github.com/dotnet-maestro-bot/msbuild merge/vs17.9-to-main
(make changes)
git commit -m "Updated PR with my changes"
git push https://github.com/dotnet-maestro-bot/msbuild HEAD:merge/vs17.9-to-main
or if you are using SSH
git checkout -b merge/vs17.9-to-main main
git pull git@github.com:dotnet-maestro-bot/msbuild merge/vs17.9-to-main
(make changes)
git commit -m "Updated PR with my changes"
git push git@github.com:dotnet-maestro-bot/msbuild HEAD:merge/vs17.9-to-main

Contact .NET Core Engineering if you have questions or issues.
Also, if this PR was generated incorrectly, help us fix it. See https://github.com/dotnet/arcade/blob/master/scripts/GitHubMergeBranches.ps1.

AR-May and others added 3 commits December 13, 2023 18:16
* Disable localization for vs17.8

* Bump version
This was added in dotnet#9193 but not usable in VS.
…otnet#9558)

Fixes dotnet#9131

Context
As described on the issue, muti-targeted builds did not import the .user file on the outer build. This change makes the outer build import the .user file.

Changes Made
Added import reference to .user file in  Microsoft.Common.CrossTargeting.targets .

Testing
Test is in SDK repo (dotnet/sdk#37192)
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

…ndencies (dotnet#9503)

Fixes dotnet#9500

Previously the code was considering all projects when building up the solution dependencies and later that was blowing up due to including projects which weren't built. This moves the logic to the same loop that already filters projects which don't build so should be (trivially) more efficient anyway.
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

Handle InvalidProjectFileExceptions as expected/user errors.

Add \n to errors logged by SimpleErrorLogger.

Fixes dotnet#9394
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

…erator (dotnet#9470)

* Pass submissionId to the BuildEventContext used by SolutionProjectGenerator

When evaluating the generated .sln.metaproj, we do pass a valid submissionId, but then we don't use it when creating the BuildEventContext used by the ProjectInstance to log evaluation events. So evaluation logging events end up having SubmissionId == -1, and thus the loggers registered with the MuxLogger in the IDE do not get the evaluation events.

Fixes dotnet#9469

* Ensure that BuildEventContext passed to LoadSolutionForBuild has a valid SubmissionId

This way Metaproject generated and other events get properly routed to MuxLogger and other loggers that care about the SubmissionId.
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

* darc update-dependencies --id 199769

This should be the GA 8.0 build.

* Add Tasks.Dataflow and Channels deps

```
darc add-dependency --name System.Threading.Channels --repo https://dev.azure.com/dnceng/internal/_git/dotnet-runtime --version 8.0.0 --commit 5535e31a712343a63f5d7d796cd874e563e5ac14 --type product

darc add-dependency --name System.Threading.Channels --repo https://dev.azure.com/dnceng/internal/_git/dotnet-runtime --version 8.0.0 --commit 5535e31a712343a63f5d7d796cd874e563e5ac14 --type product
```

* Binding redirect updates to 8.0 versions

* extend SourceBuildPrebuiltBaseline to new patterns
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

@dotnet-maestro-bot dotnet-maestro-bot requested a review from a team as a code owner December 18, 2023 22:26
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

* Combine GetProperties with UpdateProperties functionality. Remove public UpdateProperties from TelemetryBase
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

rainersigwald and others added 2 commits December 21, 2023 10:59
Fixes dotnet#9562 by triggering a full-line redraw when the length of the duration string increases.
…otnet#9557)

* Update dependencies from https://github.com/nuget/nuget.client build 6.9.0.64

NuGet.Build.Tasks
 From Version 6.9.0-preview.1.50 -> To Version 6.9.0-preview.1.64

* Update dependencies from https://github.com/dotnet/roslyn build 20231215.7

Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.23608.9 -> To Version 4.9.0-3.23615.7

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

…: Build ID 8812032 (dotnet#9495)

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8767374

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8767374

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8801027

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8801027

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8805704

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8806682

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8806775

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8806775

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8812032

* Localized file check-in by OneLocBuild Task: Build definition ID 9434: Build ID 8812032

---------

Co-authored-by: AR-May <67507805+AR-May@users.noreply.github.com>
@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

@dotnet-maestro-bot
Copy link
Contributor Author

This pull request has been updated.

This PR merges commits made on vs17.9 by the following committers:

@AR-May AR-May merged commit b59f07e into dotnet:main Dec 21, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.