Tim te Beek
Tim te Beek
hi @x80486 ! We already support a number of named styles, including [Checkstyle](https://github.com/openrewrite/rewrite/blob/8e35ef38f115502f9de5330e4b83dcbf1bac4a60/rewrite-java/src/main/java/org/openrewrite/java/style/Checkstyle.java#L28-L58), [IntelliJ](https://github.com/openrewrite/rewrite/blob/d8360ef38988c12f237ca29df0494b3a982336a9/rewrite-java/src/main/java/org/openrewrite/java/style/IntelliJ.java#L28-L47) and [Spring](https://github.com/openrewrite/rewrite/blob/bf6b1c8648a6c68fb0d1cf80673621d7314d8f9b/rewrite-java/src/main/resources/META-INF/rewrite/spring.yml#L17-L39). As a first step we could add a new named style for Eclipse to...
Hi @rcsilva83 , welcome back! The change made here might be spread across multiple repositories: https://github.com/openrewrite/rewrite-hibernate https://github.com/openrewrite/rewrite-migrate-java https://github.com/openrewrite/rewrite-spring We'd need to figure out if there's a missing parallel change to...
Thanks for diving in! The most convenient way to customizing an existing composite recipe is through the recipe builder at https://app.moderne.io/. We're working on getting that integrated into docs.openrewrite.org now...
It looks like your `JavaTemplate.builder` is missing a call to `imports`; could that factor in here? Those snippets are compiled in isolation, so they need their classpath (beyond JDK) and...
There appears to have been an issue with the test above; the JavaTemplate is missing `.imports("java.io.PrintWriter")`; this works: ```java @Issue("https://github.com/openrewrite/rewrite/issues/5187") @Test void maybeAddImportInTryWithResourcesViaJavaTemplate() { rewriteRun( spec -> spec .recipes(toRecipe(() ->...
Hi @yeikel ; Seems like a good fit for a recipe indeed! We already have the GitProvenance marker: https://github.com/openrewrite/rewrite/blob/982ad13ca1282bc9831864670f8bb800454c8e04/rewrite-core/src/main/java/org/openrewrite/marker/GitProvenance.java#L52-L60 That ought to be available when you visit a pom file...
That'd be appreciated, thanks! Would make for a nice addition to `org.openrewrite.maven.BestPractices` https://github.com/openrewrite/rewrite/blob/846412f09c45ff4b7d7295df77fcbe28bf86927d/rewrite-maven/src/main/resources/META-INF/rewrite/maven.yml#L17-L28
> The build failed because the generated code now references the deprecated `javax.annotation.Generated` annotation, rather than the `javax.annotation.processing.Generated` annotation introduced in JDK 9. > > I worked around this by...
Finally seeing some colors in the output here! 🌈
[As discussed](https://rewriteoss.slack.com/archives/C01A843MWG5/p1727705620804699?thread_ts=1725336734.753489&cid=C01A843MWG5) on [our OSS Slack](https://join.slack.com/t/rewriteoss/shared_invite/zt-nj42n3ea-b~62rIHzb3Vo0E1APKCXEA): > I've not had time to dive in again; what I think is happening here is that we do not mark the log4j-api as...