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

React to changes in .net6p7 to how source generated file paths are created #55245

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

dibarbet
Copy link
Member

there were a couple changes to file paths for source generated docs in net6p7 causing enc diagnostics in razor to fail. This must go into VS p3 since net6p7 will soon be added to VS p3.

Prior to net6p7 we looked for file paths like: "Microsoft.NET.Sdk.Razor.SourceGenerators\Microsoft.NET.Sdk.Razor.SourceGenerators.RazorSourceGenerator\_Pages_Counter_razor.cs"

In net6p7, this changed to: "Microsoft.NET.Sdk.Razor.SourceGenerators\Microsoft.NET.Sdk.Razor.SourceGenerators.RazorSourceGenerator\Pages_Counter_razor.g.cs"
Note the .g.cs and the missing underscore at the begining of the name

@dibarbet dibarbet requested a review from a team as a code owner July 29, 2021 19:48
@dibarbet dibarbet added the LSP issues related to the roslyn language server protocol implementation label Jul 29, 2021
@dibarbet dibarbet requested a review from tmat July 29, 2021 19:48
Copy link
Member

@tmat tmat left a comment

Choose a reason for hiding this comment

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

:shipit:

@dibarbet
Copy link
Member Author

@jinujoseph this is coming in for m2/ QB - EnC diagnostics in razor will be broken as soon as they finish inserting the net6p7 SDK into VS p3 without this change

@dibarbet
Copy link
Member Author

fyi @allisonchou an M2 change we can try and get in tomorrow, otherwise it should be good for qb too.

@dibarbet dibarbet merged commit eaea702 into dotnet:release/dev17.0 Jul 30, 2021
@dibarbet dibarbet deleted the fix_enc_net6p7 branch July 30, 2021 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved to merge 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