From 11776e1d6a9143e4131c44c7922b47ca4ab687fa Mon Sep 17 00:00:00 2001
From: Nanne Baars <nanne.baars@owasp.org>
Date: Thu, 5 Jan 2023 13:57:12 +0100
Subject: [PATCH] Remove explicit goal for code formatting

`mvn verify` already checks formatting, having a separate step is not necessary. We now also check Markdown files for correct formatting.
---
 .github/workflows/build.yml | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 16a6cf2b8..8d98b7a33 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -3,8 +3,6 @@ on:
   pull_request:
     paths-ignore:
       - '.txt'
-      - '*.MD'
-      - '*.md'
       - 'LICENSE'
       - 'docs/**'
   push:
@@ -16,8 +14,6 @@ on:
       - '*'
     paths-ignore:
       - '.txt'
-      - '*.MD'
-      - '*.md'
       - 'LICENSE'
       - 'docs/**'
 
@@ -47,10 +43,8 @@ jobs:
           path: ~/.m2
           key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
           restore-keys: ${{ runner.os }}-m2-
-      - name: Check code formatting
-        run: mvn --no-transfer-progress spotless:check
       - name: Build with Maven
-        run: mvn --no-transfer-progress package
+        run: mvn --no-transfer-progress verify
 
   build:
     if: github.repository == 'WebGoat/WebGoat' && github.event_name == 'push'
@@ -70,8 +64,5 @@ jobs:
               path: ~/.m2
               key: ubuntu-latest-m2-${{ hashFiles('**/pom.xml') }}
               restore-keys: ubuntu-latest-m2-
-        - name: Check code formatting
-          #Fail fast
-          run: mvn --no-transfer-progress spotless:check
         - name: Test with Maven
           run: mvn --no-transfer-progress verify