Skip to content

Share cohost code action tests with C# extension tests#12354

Merged
davidwengier merged 6 commits intodotnet:mainfrom
davidwengier:ShareCohostCodeActionsTests
Oct 21, 2025
Merged

Share cohost code action tests with C# extension tests#12354
davidwengier merged 6 commits intodotnet:mainfrom
davidwengier:ShareCohostCodeActionsTests

Conversation

@davidwengier
Copy link
Member

Copilot can't work on things it can't run tests for. This will allow it to work on code actions more effectively.

@davidwengier davidwengier requested a review from a team as a code owner October 17, 2025 04:12
@davidwengier
Copy link
Member Author

Sorry for the last couple of commits. Will get the macbook out on monday and have a proper look :)

@DustinCampbell
Copy link
Member

Are the product changes needed to share the tests?

@davidwengier
Copy link
Member Author

Not to share the code, no. I could skip the offending tests on mac/linux and follow up with fixes later. The feature might actually be broken on those platforms and we might not know, I was just too lazy last night to get out my mac and do proper work, I was guessing based on CI results from the couch :P

@davidwengier davidwengier force-pushed the ShareCohostCodeActionsTests branch from d43796b to ddca2e1 Compare October 19, 2025 04:37
@davidwengier
Copy link
Member Author

My bad, was an obvious bit of non-cross-platform-ness in the tests that I was completely blind to 🤦‍♂️

@davidwengier
Copy link
Member Author

@dotnet/razor-tooling PTAL, there are a couple of new code actions that copilot can add for us, but if it can't run tests it probably won't do a great job :)

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.

2 participants