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

Snap main to 17.14 P3 #77332

Merged
merged 531 commits into from
Feb 25, 2025
Merged

Conversation

akhera99
Copy link
Member

No description provided.

jasonmalinowski and others added 30 commits February 7, 2025 15:24
This allows us to get logs to help diagnose failing tests.
)

This allows us to get logs to help diagnose failing tests. The bulk of
this PR is refactoring out the existing logger we had for this to reuse
it better.
The .NET SDK's RID (and PlatformTarget) inference for .NET Framework only executes on Windows. As a result, building InteractiveHost32 on non-Windows produces a differently-named NuGet package. This is causing us issues in the VMR.

Set the RID explicitly like the InteractiveHost64 process does. This ensures that the resulting package name is InteractiveHost32.x86.Symbols when built on Windows or non-Windows.
* Update data section string literals spec

* Add benchmark results

* Clarify 600 bytes

* Fix typos

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

---------

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
dibarbet and others added 16 commits February 20, 2025 11:30
Removed one-off and out of support servicing branches from the publish
data.
Was causing these files to change every build on linux.
* Update to release nuget packages

In order to publish 17.13 nuget packages, we need to update a few dependencies. I updated the minimum number of packages required from our public packages, then fixed up any restore issues. For each updated package, I then went and confirmed that the dll was actually present in 17.13.0, so we should be safe from dependency loading issues. This change does not need to be inserted into VS.

* Skip tests for microsoft/vs-solutionpersistence#95

* Skip failing tests, see dotnet#77293.

(cherry picked from commit 3350036)
* wip

* wip

* wip

* wip

* wip

* wip

* wip

* remove some commented out code

* still cleaning up

* revert file

* clean up + option

* wip

* dispose of disable of intellicode line completions

* comments

* comments

* add quota exceeded checks

* fix tests

* revert file

* remove unused optional params

* last few fixes

* feedback

* fix tests

* small fix

* fix

* feedback

* feedback

* removed per comments

* remove unused usings

* feedback

* feedback

* move stuff around with new EA structure in roslyn

* wip

* feedback

* feedbacl

* fix

* feedback

* feedback

* feedback

* revert

* remove null check

* clean up

* fix

* fix tests

* last fixes

* telemetry + fixes

* last comments
@akhera99 akhera99 requested review from a team as code owners February 25, 2025 00:09
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Feb 25, 2025
Copy link
Contributor

This PR modifies public API files. Please follow the instructions at https://github.com/dotnet/roslyn/blob/main/docs/contributing/API%20Review%20Process.md for ensuring all public APIs are reviewed before merging.

@dotnet-policy-service dotnet-policy-service bot added the Needs API Review Needs to be reviewed by the API review council label Feb 25, 2025
@akhera99 akhera99 merged commit d2c3d60 into dotnet:release/dev17.14 Feb 25, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure Needs API Review Needs to be reviewed by the API review council untriaged Issues and PRs which have not yet been triaged by a lead VSCode
Projects
None yet
Development

Successfully merging this pull request may close these issues.