Add support for ExperimentalAttribute #6971
Conversation
Codecov Report
@@ Coverage Diff @@
## main #6971 +/- ##
=======================================
Coverage 96.42% 96.43%
=======================================
Files 1410 1410
Lines 335881 335956 +75
Branches 11090 11095 +5
=======================================
+ Hits 323879 323971 +92
+ Misses 9209 9194 -15
+ Partials 2793 2791 -2 |
|
Looking |
| [ID1]C.C() -> void"; | ||
|
|
||
| await VerifyNet80CSharpAdditionalFileFixAsync(source, shippedText, unshippedText, fixedUnshippedText); | ||
| } |
There was a problem hiding this comment.
Consider also testing removal of Experimental attribute (shipped file contains the experiment, but API no longer marked as experimental) and change of Experimental attribute (however I'm not sure it's a realistic/common scenario)
There was a problem hiding this comment.
Currently this will be treated as simply removing an old API and adding a new one. It's not the best, but it should work for now. We can work to improve this experience in the future based on how things go with Roslyn experiments.
|
@jcouv I'm going to submit new tests based on how this first round goes when I submit a PR with an experiment to Roslyn. |
Fixes #6759
Experimental items are shown in API files with a
[diagnosticId]prefix, wherediagnosticIdis the ID provided toExperimentalAttribute.