Tim te Beek

Results 1243 comments of Tim te Beek

In particular I like this perhaps simpler approach > If you have only one Docket — remove it and instead add properties to your application.properties: ```properties springdoc.packagesToScan=package1, package2 springdoc.pathsToMatch=/v1, /api/balance/**...

Single Docket migrations are now integrated as of: - https://github.com/openrewrite/rewrite-spring/pull/727 Hope that helps!

Thanks for the helpful example links; indeed would be a neat addition to see this fixed with a recipe.

As [discussed on the PR](https://github.com/openrewrite/rewrite-static-analysis/pull/466#issuecomment-2661604878) we're not sure there's a good safe way to limit this to just the cases where it is safe to do so, as `Arrays.asList()` returns...

Thanks for kicking this off @kthoms ; we had so far held off on a recipe like this because `List.of` returns immutable collections, whereas `Arrays.asList` returns a new mutable array...

Thanks for the offer to help @punkratz312 ! Indeed would be good to have a recipe to phase out this practice. What I can recommend here is to start a...

It looks like this test failed in a way I hadn't see before 🤔 ``` MigrateJobParameterTest > test1() FAILED java.io.UncheckedIOException: java.io.IOException: Failed to create directory /home/runner/.rewrite/classpath/.tt/org/springframework/batch/spring-batch-core at org.openrewrite.java.internal.parser.TypeTable$Reader.lambda$writeClassesDir$4(TypeTable.java:247) at org.openrewrite.java.internal.parser.TypeTable$Reader$$Lambda$483/0x00007fdfac17b4c8.accept(Unknown...

Thanks! I've triggered the CI builds again; that should only be an issue for your first contribution; after that they should start automatically.

Thanks for the ping @qwtfps ! I was out for a few days, but trying to catch up now. The latest failures now are: ``` MigrateJobParameterTest > addStringClassArguments() FAILED org.opentest4j.AssertionFailedError:...

Thanks for the suggestion & offer to help @ishish07 ! I had no idea the same was supported for properties files. Yes makes sense to do offer the same there,...