Skip to content

Revert "Remove EditorFeatures from OOP (#78069)" #78179

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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

dibarbet
Copy link
Member

@dibarbet dibarbet commented Apr 17, 2025

This change caused failed multiple failed speedometer / RPS tests (when paired with the Razor dual insertion)

Failing tests can be seen here - https://dev.azure.com/devdiv/DevDiv/_git/VS/pullrequest/629100

Validation for this run is here (no failed tests, some speedometer issues)
https://dev.azure.com/devdiv/DevDiv/_git/VS/pullrequest/629229

This reverts commit 931392e.

@ghost ghost added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Apr 17, 2025
@dibarbet dibarbet marked this pull request as ready for review April 17, 2025 18:26
@dibarbet dibarbet requested review from a team as code owners April 17, 2025 18:26
@jasonmalinowski jasonmalinowski merged commit 72266a2 into dotnet:main Apr 17, 2025
25 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Apr 17, 2025
ryzngard pushed a commit to ryzngard/roslyn that referenced this pull request Apr 17, 2025
dibarbet added a commit that referenced this pull request Apr 21, 2025
- **Reapply "Remove EditorFeatures from OOP (#78069)" (#78179)**
- **Fix missing VSIX, IVT, and PublishData changes**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants