Skip to content

Fix correctness build#83741

Merged
davidwengier merged 1 commit into
dotnet:mainfrom
davidwengier:FixBuild
May 18, 2026
Merged

Fix correctness build#83741
davidwengier merged 1 commit into
dotnet:mainfrom
davidwengier:FixBuild

Conversation

@davidwengier
Copy link
Copy Markdown
Member

@davidwengier davidwengier commented May 18, 2026

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings May 18, 2026 01:09
@davidwengier davidwengier requested a review from a team as a code owner May 18, 2026 01:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes a redundant using Microsoft.CodeAnalysis; directive that was triggering a correctness build failure (likely an unnecessary-using warning treated as error), since the file's namespace Microsoft.CodeAnalysis.Razor.Completion already brings Microsoft.CodeAnalysis into scope.

Changes:

  • Remove redundant using Microsoft.CodeAnalysis; from RazorCommitCharacter.cs.

Copy link
Copy Markdown
Contributor

@ToddGrun ToddGrun left a comment

Choose a reason for hiding this comment

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

I don't understand why this wasn't flagged in my PR that added it

@davidwengier
Copy link
Copy Markdown
Member Author

Me neither, but at least you can have a friendly squirrel on your desk now to keep you company :)

@davidwengier davidwengier enabled auto-merge May 18, 2026 02:14
@davidwengier davidwengier merged commit 861ddaf into dotnet:main May 18, 2026
26 of 27 checks passed
@davidwengier davidwengier deleted the FixBuild branch May 18, 2026 02:25
@dotnet-policy-service dotnet-policy-service Bot added this to the Next milestone May 18, 2026
@davidwengier
Copy link
Copy Markdown
Member Author

Okay, worked it out, thanks to more correctness issues after my code actions PR was merged. These come from the bootstrap correctness leg, which doesn't run on PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants