Skip to content

Commit 35a9de9

Browse files
committed
[optimise] Remove duplication of Maven phases between Test and Javadoc CI jobs
See https://github.com/eXist-db/exist/pull/4852\#discussion_r116060715
1 parent e6eecce commit 35a9de9

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

.github/workflows/ci-test.yml

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -30,17 +30,6 @@ jobs:
3030
cache: 'maven'
3131
- run: mvn -V -B dependency-check:check
3232
timeout-minutes: 60
33-
documentation:
34-
name: Javadoc
35-
runs-on: ubuntu-latest
36-
steps:
37-
- uses: actions/checkout@v3
38-
- uses: actions/setup-java@v3
39-
with:
40-
distribution: temurin
41-
java-version: ${{ env.DEV_JDK }}
42-
cache: 'maven'
43-
- run: mvn -V -B -q -T 1C install javadoc:javadoc -DskipTests -D'dependency-check.skip' -D'license.skip' --projects '!exist-distribution,!exist-installer' --also-make
4433
test:
4534
name: ${{ matrix.os }} Test
4635
strategy:
@@ -62,6 +51,9 @@ jobs:
6251
- name: Maven Test
6352
timeout-minutes: 60
6453
run: mvn -V -B verify -DtrimStackTrace=false -D'dependency-check.skip' -D'license.skip'
54+
- name: Javadoc (Linux only)
55+
if: ${{ matrix.os == 'ubuntu-latest' }}
56+
run: mvn -V -B -q -T 1C install javadoc:javadoc -DskipTests -D'dependency-check.skip' -D'license.skip' --projects '!exist-distribution,!exist-installer' --also-make
6557
- name: Maven Code Coverage
6658
if: ${{ github.ref == 'refs/heads/develop' && matrix.os == 'ubuntu-latest' }}
6759
env:

0 commit comments

Comments
 (0)