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

Merge release/dev17.13 to main #75672

Merged
merged 38 commits into from
Nov 6, 2024
Merged

Conversation

dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from release/dev17.13 into main.

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request. 🐯

Troubleshooting conflicts

Identify authors of changes which introduced merge conflicts

Scroll to the bottom, then for each file containing conflicts copy its path into the following searches:

Usually the most recent change to a file between the two branches is considered to have introduced the conflicts, but sometimes it will be necessary to look for the conflicting lines and check the blame in each branch. Generally the author whose change introduced the conflicts should pull down this PR, fix the conflicts locally, then push up a commit resolving the conflicts.

Resolve merge conflicts using your local repo

Sometimes merge conflicts may be present on GitHub but merging locally will work without conflicts. This is due to differences between the merge algorithm used in local git versus the one used by GitHub.

git fetch --all
git checkout -t upstream/merges/release/dev17.13-to-main
git reset --hard upstream/main
git merge upstream/release/dev17.13
# Fix merge conflicts
git commit
git push upstream merges/release/dev17.13-to-main --force

Cosifne and others added 20 commits September 18, 2024 15:20
…17.10

#### AI description  (iteration 1)
#### PR Classification
Code cleanup and new feature.

#### PR Summary
This pull request includes code cleanup, new tests, and the addition of new YAML templates for generating SAS tokens and federated access tokens.
- `src/Features/CSharpTest/EditAndContinue/TopLevelEditingTests.cs`: Added new tests for class implementations of interfaces with various method types.
- `src/EditorFeatures/Test2/Diagnostics/DiagnosticServiceTests.vb`: Removed the `TestTryAppendDiagnosticsForSpanAsync` test.
- `eng/common/templates-official/steps/get-delegation-sas.yml` and `eng/common/templates/steps/get-delegation-sas.yml`: Added new YAML templates for generating SAS tokens.
- `src/Features/LanguageServer/Protocol/Features/Diagnostics/EngineV2/DiagnosticIncrementalAnalyzer_GetDiagnosticsForSpan.cs`: Refactored method to use `LatestDiagnosticsForSpanGetter`.
- `src/EditorFeatures/Test/Structure/StructureTaggerTests.cs`: Added a new test to verify if statements should be collapsed.
Update STJ

----
#### AI description  (iteration 1)
#### PR Classification
Dependency update

#### PR Summary
This pull request updates the `System.Text.Json` dependency and modifies a NuGet package source URL.
- `NuGet.config`: Updated the URL for the `darc-int-dotnet-runtime` package source.
- `src/Workspaces/Core/MSBuild.BuildHost/Microsoft.CodeAnalysis.Workspaces.MSBuild.BuildHost.csproj`: Updated `System.Text.Json` package version from 8.0.4 to 8.0.5.
[release/dev17.10] Merge from internal 17.10
* Update dependencies from https://github.com/dotnet/arcade build 20240925.3

Microsoft.SourceBuild.Intermediate.arcade , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.24470.2 -> To Version 8.0.0-beta.24475.3

* Update dependencies from https://github.com/dotnet/arcade build 20241004.2

Microsoft.SourceBuild.Intermediate.arcade , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.24475.3 -> To Version 8.0.0-beta.24504.2

* Update dependencies from https://github.com/dotnet/arcade build 20241008.1

Microsoft.SourceBuild.Intermediate.arcade , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.24504.2 -> To Version 8.0.0-beta.24508.1

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Ankita Khera <[email protected]>
Also plumbs through updating the attribute throughout the various Solution and *State classes.
Use the unmapped references to update the HasSdkCodeStyleAnalyzer ProjectAttribute.
When Features analyzers are being used to replace SDK CodeStyle analyzer we need to treat them as project analyzers and not provide them with the Host's fallback options.
…analyzers.

In particular we will treat Features analyzers as project analyzers when a project is using SDK CodeStyle analyzers. This mirrors the change that redirects Features analyzers in the HostStateSets. This is necessary to keep the CompilationsWithAnalyzers in sync with the statesets.
…ase/dev17.11

Merge release/dev17.10 to release/dev17.11
…016.1 (#75538)

Microsoft.SourceBuild.Intermediate.arcade , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.24508.1 -> To Version 8.0.0-beta.24516.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
…nalyzers

When replacing CodeStyle analyzers do not provide Features analyzers fallback options
…ase/dev17.13

Merge release/dev17.12 to release/dev17.13
@akhera99 akhera99 requested a review from a team as a code owner October 31, 2024 23:57
dotnet-bot and others added 10 commits November 1, 2024 01:50
…ase/dev17.11

Merge release/dev17.10 to release/dev17.11
…ase/dev17.12

Merge release/dev17.11 to release/dev17.12
…ase/dev17.12

Merge release/dev17.11 to release/dev17.12
…ase/dev17.13

Merge release/dev17.12 to release/dev17.13
…ase/dev17.13

Merge release/dev17.12 to release/dev17.13
Fix flow of CompilerGeneratedFilesOutputPath from CPS
@jjonescz jjonescz force-pushed the merges/release/dev17.13-to-main branch from 4bb5373 to ec87a60 Compare November 4, 2024 09:35
@jjonescz jjonescz requested a review from a team as a code owner November 4, 2024 09:35
Similar to #75528.
@JoeRobich
Copy link
Member

Integration test failures may be related to this known issue. https://github.com/dotnet/core/blob/main/release-notes/9.0/known-issues.md#net-sdk-90100-rc1-will-fail-when-used-with-visual-studio-1711

@jjonescz
Copy link
Member

jjonescz commented Nov 5, 2024

Integration test failures may be related to this known issue. https://github.com/dotnet/core/blob/main/release-notes/9.0/known-issues.md#net-sdk-90100-rc1-will-fail-when-used-with-visual-studio-1711

Indeed there is a failure

Microsoft.PackageDependencyResolution.targets(266, 5): error NETSDK1060: Error reading assets file: Error loading lock file 'T:\roslyn-test\anbaudin.h1t\TestProj\obj\project.assets.json' : Could not load file or assembly 'System.Text.Json, Version=8.0.0.4, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The system cannot find the file specified.

but why it appears only in the one test Roslyn.VisualStudio.NewIntegrationTests.CSharp.CSharpRedirectFeaturesAnalyzers.DoesNotUseHostOptions_WhenEnforceCodeStyleInBuildIsTrue? Is it special somehow?

@JoeRobich
Copy link
Member

Is it special somehow?

Looking at the other integration tests. They generate their projects from the VS templates. The CSharpRedirectFeaturesAnalyzers uses a hardcoded project file so that I can set <EnforceCodeStyleInBuild>. I am going to look at switching over to using a project template then updating the project appropriately.

@dotnet-bot dotnet-bot merged commit cc477f6 into main Nov 6, 2024
28 checks passed
@jjonescz jjonescz deleted the merges/release/dev17.13-to-main branch November 6, 2024 10:14
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Nov 6, 2024
@jjonescz jjonescz modified the milestones: Next, 17.13 P2 Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants