Skip to content

Conversation

@davidwengier
Copy link
Member

@davidwengier davidwengier commented Aug 14, 2025

The Razor side of dotnet/vscode-csharp#8512
Needs dotnet/roslyn#79891 for the interface definition, and to hook things up in the misc files project
Fixes #9519
Fixes #11833

This could be done as its own PR without the source generator locator service, and it would all be the same, but also impossible to hit in real life, so I just bundled it all together.

@davidwengier davidwengier requested a review from a team as a code owner August 14, 2025 02:43
@chsienki
Copy link
Member

LGTM Modulo build errors, and assuming tests then pass.

The imports document it expected to add to was outside the project cone, which should never have worked.
@davidwengier davidwengier enabled auto-merge August 20, 2025 02:15
@davidwengier davidwengier merged commit c8b7900 into dotnet:main Aug 20, 2025
11 checks passed
@davidwengier davidwengier deleted the SourceGeneratorLocator branch August 20, 2025 02:26
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Aug 20, 2025
@davidwengier
Copy link
Member Author

/backport to release/dev18.0

@github-actions
Copy link
Contributor

Started backporting to release/dev18.0: https://github.com/dotnet/razor/actions/runs/17089789307

phil-allen-msft added a commit that referenced this pull request Aug 20, 2025
…iscellaneous files (#12126)

Backport of #12106 to release/dev18.0

/cc @davidwengier

## Customer Impact

## Regression

- [ ] Yes
- [ ] No

[If yes, specify when the regression was introduced. Provide the PR or
commit if known.]

## Testing

[How was the fix verified? How was the issue missed previously? What
tests were added?]

## Risk

[High/Medium/Low. Justify the indication by mentioning how risks were
measured and addressed.]
@RikkiGibson RikkiGibson modified the milestones: Next, 18.0 P1 Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatically hook up the source generator in Misc Files Cohost Razor and Roslyn to better share object types and object instances

3 participants