Report unused suppressions

[timotheeguerin]

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/compiler@10805

commit: b081a35

[github-actions]

All changed packages have been documented.

• ✅ @typespec/compiler

Show changes

@typespec/compiler - feature ✏️

Dim unused #suppress directives for available compiler and library diagnostics in editor scenarios.,> ,> typespec,> #suppress "deprecated" "old suppression",> model Widget {},>

[azure-sdk-automation]

You can try these changes here

🛝 Playground	🌐 Website	🛝 VSCode Extension

[markcowl]

This is super-cool

I like the gate on the diagnostic being in scope, but I think we should either turn this into an info diagnostic for now, or put it behind an emitter option. I think once we have some experience with this we can make it part of standard (without emitters) compilation.

Note: I assume the idea is that the main user of compilation without emitters is the IDE extensions, and this is a way to get this behavior without having to update the IDEs