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.
This commit is contained in:
Nanne Baars 2023-01-05 13:57:12 +01:00 committed by Nanne Baars
parent 7664625afa
commit 11776e1d6a

View File

@ -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