From 87efaa7d4c1bbc340fd79d64a82bc1553994830c Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Wed, 26 Aug 2026 11:09:07 +0300 Subject: [PATCH] chore: Check test sources with OnlyNullMarked like main sources The testCompile execution listed com.vaadin.flow.signals as an annotated package while the compile execution relied on OnlyNullMarked. Test classes share package names with the main sources, so @NullMarked in the main package-info already covers them and the package list adds nothing: both settings produce identical NullAway diagnostics across the whole test tree. --- flow-server/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/flow-server/pom.xml b/flow-server/pom.xml index c5a0d81f012..e3c2d4098a5 100644 --- a/flow-server/pom.xml +++ b/flow-server/pom.xml @@ -241,7 +241,7 @@ -J--add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED -J--add-opens=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED -J--add-opens=jdk.compiler/com.sun.tools.javac.comp=ALL-UNNAMED - -Xplugin:ErrorProne -XepDisableAllChecks -Xep:NullAway:ERROR -XepOpt:NullAway:JSpecifyMode=true -XepOpt:NullAway:AnnotatedPackages=com.vaadin.flow.signals + -Xplugin:ErrorProne -XepDisableAllChecks -Xep:NullAway:ERROR -XepOpt:NullAway:JSpecifyMode=true -XepOpt:NullAway:OnlyNullMarked=true