Tim te Beek

Results 1243 comments of Tim te Beek

@rregout I've replicate the above with a unit test in [AddManagedDependencyTest](https://github.com/openrewrite/rewrite/blob/345a8f29998d218f8d5b4e75d2d4a1e826b48a9a/rewrite-maven/src/test/java/org/openrewrite/maven/AddManagedDependencyTest.java#L29). Not sure yet why this problem occurs. I suspect this update to the Maven model might not have occurred...

Thanks! I think this might be similar to this issue right? - https://github.com/openrewrite/rewrite/issues/4240 We added some basic JSP parser support in March via - https://github.com/openrewrite/rewrite/pull/4075 Looks like we'd need to...

> found https://github.com/wjase/jsp-parser/tree/master/src/main/antlr4/com/cybernostics/jsp/parser which could be usefull for a bases for the antlr processing Nice find! One of those times where "last changed 7 years ago" feels right, and not...

Here's a proposed extension to our XML handling to support more JSP elements: - https://github.com/openrewrite/rewrite/pull/6054

Lacking detail here; why not on `Collectors`? We already have `UseStaticImports`. Does that fit the bill here? If DependencyScope is an enum (unclear from context) then this would not be...

hi! We ought to be parsing the configured timeout after this PR - https://github.com/openrewrite/rewrite/pull/4302 Note that we do not support all configuration options there, but a singular `` should work....

Thanks for the feedback. Perhaps an example file or even a PR that fixes this could help then, as I'm still a bit confused.

Do I understand correctly that you want these paragraph elements inserted through a new recipe? Please try to use the templates we provide when logging issues. It's at times hard...

Ok, then this should be relatively straightforward as a new recipe using the JavadocVisitor to insert those `` cases where absent.

We already have a recipe that does this: https://github.com/openrewrite/rewrite-static-analysis/blob/9c48fb1144f710a9672ab2dd9074ebe1b42fc73a/src/test/java/org/openrewrite/staticanalysis/RemoveUnusedPrivateMethodsTest.java#L35-L67 We can reopen if there's a reproducer unit test, or when there's more details to share based on attaching a JVM...