Greg Oledzki

Results 44 comments of Greg Oledzki

Thank you again for reporting. I've managed to simplify the test case to: ```java @Test void empty2() { rewriteRun( java( """ public class WithEmptyBlock { public void shouldNotFlipCondition() { if...

I wonder where does `maven.springframework.org` even come from? When googling, I fail to see any mention. It doesn't seem mention in Rewrite codebase either.

@crankydillo thanks for reporting this. It turns out the test case works now - as seen in #5195. So I assume the problem must've been fixed in the meantime.

We are also affected by this. I've spent some time debugging. Here's a simple repro: ``` @Command(name = "top", subcommands = {Main.Edit.class, Main.Update.class}) public class Main implements Callable { public...

I haven't been able to reproduce when running `RemoveTryCatchFailBlocks`, but what stands out is that the only `try ... catch` in this Java file is this line: https://github.com/operaton/operaton/blob/v1.0.0-beta-3/engine/src/test/java/org/operaton/bpm/engine/test/bpmn/mail/EmailSendTaskTest.java#L190 And the...

My knowledge of Kotlin is close to zero, but I am happy to help. I've just requested to join the Kotlin Lang Slack to read the context.

Marking as good-first-issue as I believe this should be quite easy for beginners to implement. Whoever does it, please come up with the description of the interface of the recipe...

- https://github.com/openrewrite/rewrite-static-analysis/pull/466

FYI, a follow-up has been merged: #703. I thought you might be interested.