search

ygojson / ygojson-tools

Tooling for the ygojson project
MIT License
1 stars 0 forks source link

Remove serializable for records one QuarkusTest does not require them #133

Closed magicDGS closed 3 weeks ago

magicDGS commented 4 months ago

On #132, we discovered that records required to be serializable to work with quarkus-tests. This is already reported to quarkus for a long time (see https://github.com/quarkusio/quarkus/issues/15892), but looks like no move have been done in that regard.

We should keep this issue open to simplify our code and remove the serializable class for our enums, as this is a requirement for test-only and should not leak into our actual code.

magicDGS commented 4 months ago

This is already fixed in Quarkus, although not releas is yet done for it. We should keep an eye on it.

magicDGS commented 4 months ago

Looks like it will be present in quarkus-3.12 (currently only on 3.11). Let's wait to upgrade when it is released

magicDGS commented 3 months ago

Still in 3.12.1 is not fixed, as the change to JBoss Marshalling was reverted. I added a comment into the original issue and I will open it again if it is not re-open.