Skip to content

Order Razor options service#54228

Closed
allisonchou wants to merge 1 commit intodotnet:mainfrom
allisonchou:OrderRazorOptionsService
Closed

Order Razor options service#54228
allisonchou wants to merge 1 commit intodotnet:mainfrom
allisonchou:OrderRazorOptionsService

Conversation

@allisonchou
Copy link
Copy Markdown
Contributor

Razor's options service should take precedence before the .editorconfig options service. Otherwise, if a user has an .editorconfig in a Razor project and the .editorconfig is configured with C# settings, those settings will override the settings from the Razor options service.

This change will not affect non-Razor scenarios, since the Razor options service only affects Razor files.

@allisonchou allisonchou requested a review from a team as a code owner June 18, 2021 22:21
@ghost ghost added the Area-IDE label Jun 18, 2021
@allisonchou allisonchou added the LSP issues related to the roslyn language server protocol implementation label Jun 18, 2021
Copy link
Copy Markdown

@NTaylorMullen NTaylorMullen left a comment

Choose a reason for hiding this comment

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

Good catch!

@allisonchou
Copy link
Copy Markdown
Contributor Author

Superseded by #54857

@allisonchou allisonchou deleted the OrderRazorOptionsService branch July 16, 2021 23:10
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