From 08216fd366fc2bfb5e31d776fb5207e5e9b3f57f Mon Sep 17 00:00:00 2001 From: "James R. Perkins" Date: Tue, 21 Nov 2023 11:50:03 -0800 Subject: [PATCH] Require the format-check before running tests. Remove the ignored-path that did not exist anyway. --- .github/workflows/maven.yml | 37 +++++++++++++++++-------------------- 1 file changed, 17 insertions(+), 20 deletions(-) diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml index 16f86d07..bce4c2de 100644 --- a/.github/workflows/maven.yml +++ b/.github/workflows/maven.yml @@ -7,8 +7,6 @@ on: push: branches-ignore: - 'dependabot/**' - paths-ignore: - - ".github/workflows/manual-*.yml" pull_request: branches: - '**' @@ -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 @@ -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 \ No newline at end of file + path: '**/server.log' \ No newline at end of file