Merge pull request #58 from juherr/features/nullaway

This pull request introduces NullAway static analysis support to improve null safety in the codebase. The main purpose is to enhance code quality by adding null annotations and restructuring imports/formatting to align with NullAway requirements.

Author: juherr

.github/workflows/main.yml

@@ -12,7 +12,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-22.04, ubuntu-24.04 ]
-        java: [ '17', '21' ]
+        java: [ '21', '24' ]
         db: [ 'mysql:8.0', 'mariadb:10.3', 'mariadb:10.4.30', 'mariadb:10.5.21', 'mariadb:10.6.14' ]
     runs-on: ${{ matrix.os }}
     services:

.github/workflows/review.yml

@@ -9,27 +9,22 @@ jobs:
         with:
           fetch-depth: 0
 
-      - name: Set up Java 17
+      - name: Set up Java 21
         uses: actions/setup-java@v4
         with:
-          java-version: 17
           distribution: 'temurin'
+          java-version: '21'
           cache: maven
 
-      - name: Check with Maven
+      - name: Check license headers
         run: ./mvnw -B -V com.mycila:license-maven-plugin:5.0.0:check
-  checkstyle:
+  style-check:
     runs-on: 'ubuntu-latest'
     steps:
       - uses: actions/checkout@v5
       - uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
-          java-version: '17'
-      - name: Run Checkstyle
-        uses: nikitasavinov/checkstyle-action@master
-        with:
-          checkstyle_config: './build-tools/src/main/resources/steve/checkstyle.xml'
-          workdir: '.'
-          reporter: 'github-pr-check'
-          tool_name: 'checkstyle'
+          java-version: '21'
+      - name: Check code style
+        run: ./mvnw spotless:check

.mvn/jvm.config

@@ -0,0 +1,11 @@
+--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.model=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.processing=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED
+--add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
+--add-opens jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED
+--add-opens jdk.compiler/com.sun.tools.javac.comp=ALL-UNNAMED

build-tools/pom.xml

@@ -0,0 +1,3 @@
+config.stopBubbling = true
+lombok.addNullAnnotations = jspecify
+lombok.addLombokGeneratedAnnotation = true