Skip to content

Commit

Permalink
Require the format-check before running tests. Remove the ignored-pat…
Browse files Browse the repository at this point in the history
…h that did not exist anyway.
  • Loading branch information
jamezp committed Nov 21, 2023
1 parent 6a2304b commit 08216fd
Showing 1 changed file with 17 additions and 20 deletions.
37 changes: 17 additions & 20 deletions .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@ on:
push:
branches-ignore:
- 'dependabot/**'
paths-ignore:
- ".github/workflows/manual-*.yml"
pull_request:
branches:
- '**'
Expand All @@ -19,8 +17,23 @@ concurrency:
cancel-in-progress: true

jobs:
build:
format-check:
runs-on: ubuntu-latest
timeout-minutes: 30

steps:
- uses: actions/checkout@v4
- name: Set up JDK 11
uses: actions/setup-java@v3
with:
java-version: 11
distribution: 'temurin'
cache: 'maven'
- name: Validate Formatting
run: |
mvn -B validate -Pformat-check -Denforcer.skip=true
build:
needs: format-check
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
Expand All @@ -47,20 +60,4 @@ jobs:
if: failure()
with:
name: server-logs-${{ matrix.os }}-${{ matrix.java }}
path: '**/server.log'

format-check:
runs-on: ubuntu-latest
timeout-minutes: 30

steps:
- uses: actions/checkout@v4
- name: Set up JDK 11
uses: actions/setup-java@v3
with:
java-version: 11
distribution: 'temurin'
cache: 'maven'
- name: Validate Formatting
run: |
mvn -B validate -Pformat-check -Denforcer.skip=true
path: '**/server.log'

0 comments on commit 08216fd

Please sign in to comment.