Jonathan Schnéider

@sambsnyd A quick examination of how java treats this shows that this would not in fact be very hard: ```java @SuppressWarnings({"SwitchStatementWithTooFewBranches", "ConstantConditions", "UnusedLabel", "TextLabelInSwitchStatement"}) public class DanglingSwitchLabel { @Test void...

The correct answer is somewhere in the middle... The most "do-no-harm" approach initially would be to check if there is a potential null on the left side and replace with...

@sambsnyd Dataflow analysis is in development

@sambsnyd Dataflow analysis is in development.

Partially solved by #1537. Now just need one more module `rewrite-all` (?) that takes first order dependencies on all modules.

@shanman190 We'd want a separate `rewrite-all` from this repo that also includes all recipe jars.

Just so I don't lose this... this is the OpenRewrite YAML we're using to generate this change that's got some edits from the vanilla JUnit 4->5 migration to account for...

> What we've done with those tests which previously were automatically skipped by junit4 rules was just simply convert into IT tests and add junit5 tags. Sounds like a sensible...

Just wanted you to know I haven't forgotten about you ;). Will come back with an updated PR.

+1 for this, as with this support we can use instance tags as in Prometheus label remapping rather than having to pack these tags into common tags shipped from the...