Skip to content

Add test coverage for completion in component parameters with leading @#12405

Merged
davidwengier merged 3 commits into
mainfrom
copilot/fix-blazor-completion-issue
Oct 28, 2025
Merged

Add test coverage for completion in component parameters with leading @#12405
davidwengier merged 3 commits into
mainfrom
copilot/fix-blazor-completion-issue

Conversation

Copilot AI commented Oct 28, 2025

Copy link
Copy Markdown

Fixes #8442

Summary

Successfully added comprehensive test coverage for issue #8442 regarding completion in Blazor component parameters with leading @ character. Updated all WorkItem attributes to reference the correct issue number per code review feedback.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix completion list issue for Blazor component parameters Add test coverage for completion in component parameters with leading @ Oct 28, 2025
Copilot AI requested a review from davidwengier October 28, 2025 10:44

Copilot AI commented Oct 28, 2025

Copy link
Copy Markdown
Author

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • mobile.events.data.microsoft.com
    • Triggering command: /usr/share/dotnet/dotnet exec --runtimeconfig /home/REDACTED/work/razor/razor/artifacts/bin/Microsoft.VisualStudioCode.RazorExtension.Test/Debug/net9.0/Microsoft.VisualStudioCode.RazorExtension.Test.runtimeconfig.json --depsfile /home/REDACTED/work/razor/razor/artifacts/bin/Microsoft.VisualStudioCode.RazorExtension.Test/Debug/net9.0/Microsoft.VisualStudioCode.RazorExtension.Test.deps.json /home/REDACTED/work/razor/razor/artifacts/bin/Microsoft.VisualStudioCode.RazorExtension.Test/Debug/net9.0/testhost.dll --port 38299 --endpoint 127.0.0.1:038299 --role client --parentprocessid 6482 --telemetryoptedin false (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@davidwengier davidwengier marked this pull request as ready for review October 28, 2025 11:01
@davidwengier davidwengier requested a review from a team as a code owner October 28, 2025 11:01

@ToddGrun ToddGrun left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

:shipit:

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.

No completion list after dot when specifying Blazor component parameter value with C# expression that leads with @

3 participants