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

Remove IRazorDocumentOptionsService inheritance interface #54047

Merged
merged 1 commit into from
Jun 11, 2021

Conversation

allisonchou
Copy link
Contributor

Follow-up from #53879.
I made an error since IDocumentService is unneeded (since we go through MEF now instead) and also inaccessible from the Razor side. 🤦‍♀️

@allisonchou allisonchou added Area-IDE LSP issues related to the roslyn language server protocol implementation labels Jun 11, 2021
@allisonchou allisonchou requested a review from a team as a code owner June 11, 2021 21:14
@allisonchou allisonchou enabled auto-merge (squash) June 11, 2021 21:58
@allisonchou allisonchou merged commit ec902d6 into main Jun 11, 2021
@ghost ghost added this to the Next milestone Jun 11, 2021
@allisonchou allisonchou deleted the RemoveRazorOptionsInterface branch June 11, 2021 22:36
333fred added a commit that referenced this pull request Jun 14, 2021
…ures/interpolated-string

* upstream/main: (95 commits)
  Update official build number in separate job
  Update Language Feature Status.md (#54015)
  Remove IRazorDocumentOptionsService inheritance interface (#54047)
  Fix comment
  Simplify
  Do not create a cache field for lambda if it depends on caller's type argument (#44939)
  Documentation
  Documentation
  Documentation
  Update test impls
  Just pass null
  Pull diagnostics should just request from the doc, not the whole project.
  Add test plan for file-scoped namespace (#54003)
  Add source build to official build
  Improved nullable 'is' analysis (#53311)
  Multi session service (#53762)
  Resolve Versions.props conflicts
  Revert "Revert "Require partial method signatures to match" (47576) (#47879)" (#53352)
  Broaden enforcement on prototype marker (#53886)
  Update Language Feature Status.md (#53926)
  ...
@RikkiGibson RikkiGibson modified the milestones: Next, 17.0.P2 Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE LSP issues related to the roslyn language server protocol implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants