Skip to content

Remove IRazorDocumentOptionsService inheritance interface#54047

Merged
allisonchou merged 1 commit intomainfrom
RemoveRazorOptionsInterface
Jun 11, 2021
Merged

Remove IRazorDocumentOptionsService inheritance interface#54047
allisonchou merged 1 commit intomainfrom
RemoveRazorOptionsInterface

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