rewrite

rewrite copied to clipboard

## What's changed? Prevent an NPE by adding a null check, and changing the API response to be nullable to handle it downstream. ## What's your motivation? A community member...

timtebeek

Recipe UpgradeSpringBoot_3_3 misses upgrading wro4j-maven-plugin and the latest version of jakarta.xml.bind-api when the origin is Java8 and Spring Boot 1.5.X

3

## What version of OpenRewrite are you using? I am using: - Maven/Gradle plugin 5.34.0 - rewrite-spring 5.13.2 ## How are you running OpenRewrite? I'm using maven plugin on my...

mgvinuesa

This happens on OpenRewrite 8.28.1 `rewriteRun`-tests on a recipe that rewrites `J.MethodInvocation` arguments of type `Map` throw ```shell java.lang.IllegalStateException: LST contains missing or invalid type information MethodInvocation->Block->MethodDeclaration->Block->ClassDeclaration->CompilationUnit /*~~(MethodInvocation type is...

Philzen

ChangeType adds unnecessary import on fully qualified parameterized type.

1

```java @Test void test() { rewriteRun( java( """ package a; public class A1 { } """ ), java( """ public class B { public void method(a.A1 a) { } }...

traceyyoshima

This happens on OpenRewrite 8.28.1 A JavaTemplate string such as `#{arg:any(java.util.Map)}` will not match the argument `Map.of("key", new Object[] {1,2,3})` for example. ### Workaround Changing the template string to a...

Philzen

Draft - Added tests to UpgradeTransitiveDependencyVersionTest - Demonstrates potential dependencyManagement logic bug when versions are set via Maven property

1

## What's changed? Adds tests to UpgradeTransitiveDependencyVersionTest. Two demonstrate current behaviors and are added to prevent regression and demonstrate to the reader what the recipe can do. The third demonstrates...

gjd6640

`maybeAddImport` adds import although all references are fully qualified

1

I developing a recipe with OpenRewrite v8.27.1 and have this test: ```java @Test void isMigratedToMethods_whenFullyQualified() { // language=java rewriteRun(java( """ package de.foo.bar; @org.testng.annotations.Test public class BazTest { public void shouldDoStuff()...

Philzen

Simplify code in `ReloadableJavaParserVisitor#visitVariables()`

1

knutwannheden

add clang-format style

1

## What problem are you trying to solve? Format the code with [clang-format](https://github.com/autostyle/clang-format) style ## Describe the solution you'd like create a new style that will use ".clang-format" file from...

samuelfac

Groovy Enum is throwing "Failed to parse" "UnsupportedOperationException: enum fields are not implemented."

2

## What version of OpenRewrite are you using? I am using https://github.com/gtiwari333/openrewrite-groovy-bug/blob/main/init.gradle - org.openrewrite.recipe:rewrite-recipe-bom - 2.1.20 - org.openrewrite:rewrite-gradle-plugin - latest.release - groovy 4.0.21 - jdk 17/21 - gradle 8.8 ##...

gtiwari333