Skip to content

Revert "Revert "Make solution options global""#59567

Merged
tmat merged 1 commit intomainfrom
revert-59537-revert-59168-MakeSolutionOptionsGlobal
Feb 15, 2022
Merged

Revert "Revert "Make solution options global""#59567
tmat merged 1 commit intomainfrom
revert-59537-revert-59168-MakeSolutionOptionsGlobal

Conversation

@tmat
Copy link
Copy Markdown
Member

@tmat tmat commented Feb 15, 2022

Reverts #59537

@tmat tmat requested review from a team as code owners February 15, 2022 21:31
@tmat tmat requested a review from a team February 15, 2022 21:31
@ghost ghost added the Area-IDE label Feb 15, 2022
@RikkiGibson RikkiGibson self-requested a review February 15, 2022 21:32
@RikkiGibson RikkiGibson self-assigned this Feb 15, 2022
Copy link
Copy Markdown
Member

@RikkiGibson RikkiGibson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we want to take this as soon as it's ready--no need for a separate validation run this time. It appears that Razor is broken until we get this into VS main.

@tmat
Copy link
Copy Markdown
Member Author

tmat commented Feb 15, 2022

razor is broken? :-o

@tmat tmat merged commit 0098764 into main Feb 15, 2022
@ghost ghost added this to the Next milestone Feb 15, 2022
@tmat tmat deleted the revert-59537-revert-59168-MakeSolutionOptionsGlobal branch February 15, 2022 23:06
@RikkiGibson RikkiGibson modified the milestones: Next, 17.2.P2 Mar 1, 2022
333fred added a commit to 333fred/roslyn that referenced this pull request Mar 4, 2022
This was removed in dotnet#59567, just before the vscode API went stable and O# could finally take advantage of it. This adds the EA back.

I also added Joey and I to the codeowners file so we can be aware of any future changes.
333fred added a commit to 333fred/roslyn that referenced this pull request Mar 4, 2022
This was removed in dotnet#59567, just before the vscode API went stable and O# could finally take advantage of it. This adds the EA back.

I also added Joey and I to the codeowners file so we can be aware of any future changes.
333fred added a commit that referenced this pull request Mar 4, 2022
This was removed in #59567, just before the vscode API went stable and O# could finally take advantage of it. This adds the EA back.

I also added Joey and I to the codeowners file so we can be aware of any future changes.

Co-authored-by: Joey Robichaud <joseph.robichaud@microsoft.com>
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