Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report diagnostics for unused "using" so that they will be shown properly in ide #5453

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

RodgeFu
Copy link
Contributor

@RodgeFu RodgeFu commented Dec 26, 2024

When "using" and "import" are not needed, dim them and provide quickfix in IDE. (Extracted from the PR #5146 for the unused using part)

Fixes: #3373

@RodgeFu RodgeFu added the ide Issues for VS, VSCode, Monaco, etc. label Dec 26, 2024
@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 26, 2024

All changed packages have been documented.

  • @typespec/compiler
  • @typespec/http-server-csharp
  • @typespec/http-specs
  • @typespec/http
  • @typespec/openapi
  • @typespec/openapi3
  • @typespec/rest
  • @typespec/streams
  • @typespec/xml
Show changes

@typespec/compiler - feature ✏️

Support diagnostics for unused using statement

@typespec/http-specs - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/http - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/openapi - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/openapi3 - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/rest - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/streams - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/xml - internal ✏️

Fix test for diagnostics of unused using statement

@typespec/http-server-csharp - internal ✏️

Fix test for diagnostics of unused using statement

@azure-sdk
Copy link
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 📚 Next docs 🛝 VSCode Extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ide Issues for VS, VSCode, Monaco, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[IDE] For unused using statement, strike thru with yellow line and offer a 💡 quick fix to remove used ones
3 participants