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

OmniSharp options #58208

Merged
merged 6 commits into from
Dec 9, 2021
Merged

OmniSharp options #58208

merged 6 commits into from
Dec 9, 2021

Conversation

tmat
Copy link
Member

@tmat tmat commented Dec 9, 2021

Further options refactoring focused on options set by OmniSharp.

Adds External Access APIs that allow OmniSharp to specify options directly.

@tmat tmat requested review from a team as code owners December 9, 2021 00:38
@ghost ghost added the Needs UX Triage label Dec 9, 2021
@tmat
Copy link
Member Author

tmat commented Dec 9, 2021

FYI @jasonmalinowski

@tmat tmat merged commit f473056 into dotnet:main Dec 9, 2021
@ghost ghost added this to the Next milestone Dec 9, 2021
@tmat tmat deleted the OmniSharpOptions branch December 9, 2021 02:50
333fred added a commit to 333fred/roslyn that referenced this pull request Dec 10, 2021
…rations

* upstream/main: (396 commits)
  Update several ExpressionCompiler unit tests for inferred delegate types (dotnet#58203)
  Avoid calculating inferred delegate type unnecessarily in conversions and type inference (dotnet#58115)
  OmniSharp options (dotnet#58208)
  Fix generator caching in compiler server (dotnet#57177)
  Clarify use of null as an initialized value
  BoundDecisionDag.Rewrite - Avoid capturing the replacement map (dotnet#58137)
  Address feedback
  Add regex parser tests
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add missing delegate casts (dotnet#58170)
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  ...
tmat added a commit that referenced this pull request Dec 10, 2021
tmat added a commit to tmat/roslyn that referenced this pull request Dec 15, 2021
@Cosifne Cosifne modified the milestones: Next, 17.1.P3 Jan 5, 2022
@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.

4 participants