Tim te Beek

Hi @vudayani ; good to see you here again. Do I understand correct that you landed here from following how rewrite-spring's `ChangeSpringPropertyKey` uses the `ChangePropertyKey` defined here? Since your title...

Still catch up from last week; was great to meet briefly at Devoxx! Indeed seems best to make the existing ParentPomInsight recursive as well; judging by [how that's used](https://github.com/search?q=org%3Aopenrewrite%20ParentPomInsight&type=code) we...

Hi! Thanks for logging your findings & the patience before a reply. We've pushed a change in 5.40.2 that might change the behavior that you're seeing here: - https://github.com/openrewrite/rewrite-maven-plugin/releases/tag/v5.40.2 -...

Oh wow, thanks for the very detailed analysis and fix! Should be a great quality of life improvement to shave this much of the build time, even with tests in...

Did a first pass through (still from crappy airport wifi), but it looks good. I've tried to swap out the pom used in `nonMavenCentralRepository()` as that had a long tail...

Thanks once again @DidierLoiseau ! I'll go ahead and merge this iteration already, and we can iterate on some of the other proposals. Great that you noticed and explored this...

Following the merge Gradle Develocity's list of slowest test is indeed looking a lot better: https://ge.openrewrite.org/s/lngagjnyfjai6/tests/slowest-tests Test | Outcome | Total time -|-|- shouldNotAddToDependencyManagement() org.openrewrite.maven.ChangeParentPomTest :rewrite-maven:test | PASSED | 21.449s...

hi yes we recently added this best practices recipe for JSpecify, both to migrate away from a few existing ones, as well as to add in any potentially missing annotations:...

Hi @gsmet ! Thanks for the additional context! I think the Jenkins folks have similar requirements, as seen on this existing PR: - https://github.com/openrewrite/rewrite/pull/4404 Haven't had time to review that...

Thanks for the helpful reproducer!