Apply suggested changes

Author: GabrielFleischer

.github/workflows/build.yml

@@ -23,15 +23,15 @@ jobs:
       id-token: write  # Required for Vault OIDC authentication
       contents: write  # Required for repository access and tagging
     outputs:
-      build-number: ${{ steps.build.outputs.BUILD_NUMBER }}
-      deployed: ${{ steps.build.outputs.deployed }}
+      build-number: ${{ steps.build-step.outputs.BUILD_NUMBER }}
+      deployed: ${{ steps.build-step.outputs.deployed }}
     steps:
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
       - uses: jdx/mise-action@5ac50f778e26fac95da98d50503682459e86d566 # v3.2.0
         with:
           version: 2025.7.12
       - uses: SonarSource/ci-github-actions/build-gradle@v1
-        id: build
+        id: build-step
         with:
           deploy-pull-request: true
           artifactory-reader-role: private-reader    # Override for public repo using private access
@@ -53,14 +53,14 @@ jobs:
         item:
           - { suite: "plugin", sq_version: "DEV", submodules: false}
           - { suite: "plugin", sq_version: "LATEST_RELEASE", submodules: false}
-          - { suite: "ruling", sq_version: "LATEST_RELEASE", submodules: true}
+          - { suite: "ruling", sq_version: "LATEST_RELEASE" }
     name: "QA Tests [${{ matrix.item.suite }}] - SQ : ${{ matrix.item.sq_version }}"
     steps:
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
         with:
-          submodules: ${{ matrix.item.submodules }}
+          submodules: ${{ matrix.item.suite == "ruling" }}
       - name: Remove kotlin ITs as they are not relevant for ruling tests (and cause issues with wrapper validation)
-        if: ${{ matrix.item.submodules }}
+        if: ${{ matrix.item.suite == "ruling" }}
         run: rm -r ./its/sources/kotlin
       - uses: jdx/mise-action@5ac50f778e26fac95da98d50503682459e86d566 # v3.2.0
         with: