Remove unnecessary cache by k163377 · Pull Request #628 · FasterXML/jackson-module-kotlin (original) (raw)

@k163377 I did not set it up, but Maven plugin "japicmp-maven-plugin" was recently set up to guard against API compatibility changes by maintainers (I think @dinomite remembers this) -- alas, release notes haven't been kept up to date so I don't see the PR/issue number.
But it's this in pom.xml:

            <plugin>
                <groupId>com.github.siom79.japicmp</groupId>
                <artifactId>japicmp-maven-plugin</artifactId>

The idea being it flags anything that even theoretically breaks backwards-compatibility -- and removal of a protected field is theoretically such thing (since someone could have sub-classed class, referenced field).
In practice there are cases where such changes should be made, I think; and for that we have to add exclusions to basically tell "that's ok, don't worry about the change".
These are added under:

                    <parameter>
                        <breakBuildOnBinaryIncompatibleModifications>true</breakBuildOnBinaryIncomp\
atibleModifications>
                        <breakBuildOnSourceIncompatibleModifications>true</breakBuildOnSourceIncomp\
atibleModifications>
                        <excludes>
                           <exclude>com.fasterxml.jackson.module.kotlin.ValueClassBoxSerializer</exclude>
                           ....
                        </excludes>

so you will probably need to add one or more excludes. I went and removed ones that were left over from 2.13 -> 2.14 cases.