Disable not null instrumenter plugin as it breaks GuavaTest (#507) fixes

#509
OpenHFT · Jan 29, 2024 · ceb2436 · ceb2436
1 parent 5771856
commit ceb2436
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/pom.xml b/pom.xml
@@ -252,6 +252,22 @@
         </resources>
 
         <plugins>
+            <!-- The parent POM defines this plugin and activates it in a profile. Unfortunately this plugin will generate
+            code at @NotNull call-sites that throws IllegalArgumentException rather than NullPointerException. This will
+            break GuavaTest. Chronicle have raised this issue before with the maintainers of the plugin. See:
+            https://github.com/osundblad/intellij-annotations-instrumenter-maven-plugin/issues/53
+            -->
+            <plugin>
+                <groupId>se.eris</groupId>
+                <artifactId>notnull-instrumenter-maven-plugin</artifactId>
+                <version>1.1.1</version>
+                <configuration>
+                    <notNull>
+                        <!-- Not ideal but this was the easiest way to effectively disable this plugin -->
+                        <param>Ignore</param>
+                    </notNull>
+                </configuration>
+            </plugin>
             <plugin>
                 <groupId>net.openhft</groupId>
                 <artifactId>binary-compatibility-enforcer-plugin</artifactId>