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

Revert "Remove dependency on EditorFeatures from Remote.ServiceHub project - take 2" #59242

Merged
merged 1 commit into from
Feb 4, 2022

Conversation

RikkiGibson
Copy link
Contributor

@RikkiGibson RikkiGibson commented Feb 3, 2022

@RikkiGibson
Copy link
Contributor Author

RikkiGibson commented Feb 4, 2022

@tmat, I must apologize for the inconvenience here, but I didn't get enough runway here to coordinate an insertion with TypeScript. I need to merge this revert for now. We can revisit in 17.2-preview2.

Changes which require dual insertions need to be coordinated closely with infraswat, at minimum by tagging @dotnet/roslyn-infrastructure on GitHub or ideally by identifying the current tiger--you can always ask in the InfraSwat Teams channel. and messaging them in advance.

@RikkiGibson RikkiGibson changed the base branch from main to release/dev17.2 February 4, 2022 07:38
@RikkiGibson RikkiGibson merged commit 8d81aa5 into release/dev17.2 Feb 4, 2022
@RikkiGibson RikkiGibson deleted the revert-59147-EditorFeatures2 branch February 4, 2022 07:38
@tmat
Copy link
Member

tmat commented Feb 4, 2022

@RikkiGibson My bad, I probably accidentally hit "automerge". As stated in the description this needs to wait for Type Script.

tmat added a commit that referenced this pull request Feb 5, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Feb 5, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Feb 7, 2022
tmat added a commit that referenced this pull request Feb 8, 2022
tmat added a commit that referenced this pull request Feb 8, 2022
tmat added a commit that referenced this pull request Feb 9, 2022
tmat added a commit that referenced this pull request Feb 9, 2022
tmat added a commit that referenced this pull request Feb 10, 2022
tmat added a commit that referenced this pull request Feb 10, 2022
tmat added a commit that referenced this pull request Feb 11, 2022
…ake 3 (#59316)

* Revert "Revert "Remove dependency on EditorFeatures from Remote.ServiceHub project - take 2 (#59147)" (#59242)"

This reverts commit 8d81aa5.

* Move GoToBaseService to Features

* Remove unnecessray PackageRefs

* Fixes
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.

2 participants