Skip to content

Fix various formatting issues with lambdas in generated code#6245

Merged
davidwengier merged 3 commits intodotnet:mainfrom
davidwengier:FormatInputSelectAndCascadingValue
May 26, 2022
Merged

Fix various formatting issues with lambdas in generated code#6245
davidwengier merged 3 commits intodotnet:mainfrom
davidwengier:FormatInputSelectAndCascadingValue

Conversation

@davidwengier
Copy link
Copy Markdown
Member

@davidwengier davidwengier commented Apr 4, 2022

Fixes #6158
Fixes #5693
Fixes #6001
Fixes #5676

Test only PR, these are all fixed by the Roslyn bump, because dotnet/roslyn#60437 is fixed.

@NTaylorMullen
Copy link
Copy Markdown

Heart stopped until I saw the actual files changed 😆
image

@NTaylorMullen
Copy link
Copy Markdown

So is the gist here that our TagHelper test files were straight up wrong?

@davidwengier
Copy link
Copy Markdown
Member Author

So is the gist here that our TagHelper test files were straight up wrong?

Wrong is debatable, but certainly out of date. Couldn't use InputSelect in a test because the Common.GloballyQualifiedTypeName info was missing (IIRC)

@NTaylorMullen
Copy link
Copy Markdown

Wrong is debatable, but certainly out of date. Couldn't use InputSelect in a test because the Common.GloballyQualifiedTypeName info was missing (IIRC)

Ahhh ok ya that makes sense and that was the TagHelper that exhibited the issue

@davidwengier davidwengier force-pushed the FormatInputSelectAndCascadingValue branch from 156162b to 9dae332 Compare May 24, 2022 07:03
@davidwengier davidwengier force-pushed the FormatInputSelectAndCascadingValue branch from 9dae332 to e94de3e Compare May 24, 2022 07:05
@davidwengier davidwengier marked this pull request as ready for review May 24, 2022 07:05
@davidwengier davidwengier merged commit 117cb0a into dotnet:main May 26, 2022
@davidwengier davidwengier deleted the FormatInputSelectAndCascadingValue branch May 26, 2022 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants