Skip to content

Conversation

@jkstrauss
Copy link

This a first attempt at updating vavr-jackson for use to use Jackson 3. I was able to get all tests to run successfully locally.

@jkstrauss jkstrauss requested a review from pivovarit as a code owner December 8, 2025 19:57
```java
ObjectMapper mapper = new ObjectMapper();
mapper.registerModule(new VavrModule());
ObjectMapper mapper = new ObjectMapper().rebuild().addModule(new VavrModule()).build();
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There might be a better way than using rebuild, but this was the quickest way without changing the code significantly. This strategy was employed throughout the PR.

import static tools.jackson.core.JsonToken.VALUE_NULL;

abstract class ArrayDeserializer<T> extends ValueDeserializer<T> implements ContextualDeserializer {
abstract class ArrayDeserializer<T> extends VavrValueDeserializer<T> {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The local ValueDeserializer/ValueSerializer was renamed due to a naming conflict with Jackson 3's classes of the same names.

@jkstrauss jkstrauss mentioned this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant