Skip to content

Conversation

@tmat
Copy link
Member

@tmat tmat commented Jan 25, 2022

EditorFeatures is client code and should not be referenced by remote services.
Moves types down from EditorFeatures to Features.

@tmat tmat requested review from a team as code owners January 25, 2022 18:34
@tmat tmat requested a review from a team January 25, 2022 18:34
@ghost ghost added the Area-IDE label Jan 25, 2022
@ghost ghost added the Needs UX Triage label Jan 25, 2022
@tmat
Copy link
Member Author

tmat commented Jan 25, 2022

@ryzngard ptal

@tmat tmat enabled auto-merge (squash) January 27, 2022 01:53
@tmat tmat merged commit 3b32fc3 into dotnet:main Jan 27, 2022
@ghost ghost added this to the Next milestone Jan 27, 2022
tmat added a commit that referenced this pull request Jan 28, 2022
333fred added a commit to YairHalberstadt/roslyn that referenced this pull request Jan 28, 2022
…ess-instance-members

* upstream/main: (669 commits)
  Fix 'hasStaticConstructor' check in MethodCompiler (dotnet#59116)
  Update dependencies from https://github.com/dotnet/arcade build 20220127.8 (dotnet#59134)
  Update StreamJsonRpc (dotnet#59073)
  Resources
  Filter cancellation exceptions in generator driver (dotnet#58843)
  Revert "Remove dependency on EditorFeatures from Remote.ServiceHub project (dotnet#59059)"
  Strings
  Inline
  Convert to switch expression
  Explicitly test empty string case.
  Fix comment
  Simplify test code
  Run all
  Add tests
  Delete test generator
  Add support for specifying server in tests
  Remove options
  review feedback
  Format document after each provider (dotnet#59091)
  [main] Update dependencies from dotnet/arcade (dotnet#59015)
  ...
tmat added a commit to tmat/roslyn that referenced this pull request Jan 28, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Feb 2, 2022
@RikkiGibson RikkiGibson modified the milestones: Next, 17.2.P1 Feb 4, 2022
@tmat tmat deleted the EditorFeatures branch February 7, 2022 22:44
@ryzngard ryzngard added UX Review Not Required UX Review Not Required and removed Needs UX Triage labels Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants