Tim te Beek

## What's changed? Add first of Refaster style recipes to replace Guava with Java standard library. ## What's your motivation? Show Refaster style recipes & test approach, to allow easier...

`spring-cloud-dependencies` upgrade drops `commons-collections4` version

9

As [reported on Slack](https://rewriteoss.slack.com/archives/C01A843MWG5/p1721726228694759) by @wabrit > I have a composite recipe which makes some changes to a POM; the source POM declares a dependency on org.apache.commons:commons-collections4 which is expressed...

## What problem are you trying to solve? `HttpComponentsClientHttpRequestFactory.setReadTimeout` is removed in Spring 6.1.0 (spring boot 3.2.0). ## What precondition(s) should be checked before applying this recipe? Uses method. ##...

Migrate `WebMvcTags` to `ServerRequestObservationConvention`

2

## What problem are you trying to solve? [WebMvcTags was deprecated in 3.0](https://docs.spring.io/spring-boot/docs/3.1.0/api/org/springframework/boot/actuate/metrics/web/servlet/WebMvcTags.html), for removal in 3.2 in favor of [ServerRequestObservationConvention](https://docs.spring.io/spring-framework/docs/6.0.9/javadoc-api/org/springframework/http/server/observation/ServerRequestObservationConvention.html). The [3.0 release notes](https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.0-Release-Notes) merely mention: > RestTemplateExchangeTagsProvider, WebClientExchangeTagsProvider,...

## What problem are you trying to solve? We want to achieve a complete migration coming from JMockit going to Mockito. It seems we don't yet cover either the `@Tested`...

Add missing nullable annotation on method declaration return type

1

## What problem are you trying to solve? Nullability annotations on method return types help tools and devs to better understand and reason about the code. Folks might forget to...

Verify markers are placed on `Yaml.Sequence.Entry`, but not printed

1

Add test in response to a question that came in via email, about markers not shown in output. Turns out we do not print `SearchResult` markers on `Yaml.Sequence.Entry`, but they...

Do not change package when non-recursive

5

Add unit test as reported via Slack by @BhavanaPidapa; update precondition to evaluate recursive boolean for package declaration too.

Standardize nullability annotations

12

There's no shortage of nullability annotations; we even have our own, which are [not always consistently applied](https://github.com/openrewrite/rewrite/pull/2835). Lombok has a decent [list of mostly-interchangeable nullability annotations](https://github.com/projectlombok/lombok/blob/v1.18.26/src/core/lombok/core/configuration/NullAnnotationLibrary.java#L64), although they are [missing...

## What problem are you trying to solve? Apart from the most recent version, we might also be interested in capturing when that dependency was last updated. This then allows...