Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
use shared gh action
  • Loading branch information
olamy authored May 12, 2022
commit fd87331c1505e018dab008c86a6d0378793053a9
43 changes: 12 additions & 31 deletions .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,37 +16,18 @@
# under the License.

name: GitHub CI

on: [push, pull_request]
on:
push:
pull_request:

jobs:
build:

strategy:
matrix:
os: [ubuntu-latest,windows-latest, macOS-latest]
java: [8, 11, 14, 15]
fail-fast: false

runs-on: ${{ matrix.os }}

steps:
- name: Checkout
uses: actions/checkout@v3.0.2

- name: Set up cache for ~./m2/repository
uses: actions/cache@v3.0.2
with:
path: ~/.m2/repository
key: maven-${{ matrix.os }}-java${{ matrix.java }}-${{ hashFiles('**/pom.xml') }}
restore-keys: |
maven-${{ matrix.os }}-java${{ matrix.java }}-
maven-${{ matrix.os }}-
- name: Set up JDK
uses: actions/setup-java@v3
with:
distribution: 'adopt'
java-version: ${{ matrix.java }}

- name: Build with Maven
run: mvn install javadoc:javadoc -e -B -V
build:
name: Build it
uses: codehaus-plexus/.github/.github/workflows/maven.yml@master
with:
jdk-fast-fail-build: '11'
jdk-matrix: '["11", "17"]'
jdk-distribution-matrix: '["zulu", "temurin", "microsoft", "liberica"]'
os-matrix: '["ubuntu-latest","windows-latest", "macOS-latest"]'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the same as default

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default has more jdks

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@slawekjaranowski And default in master are changed without PRs neither asking around

maven_args: 'install javadoc:javadoc -e -B -V -fae '
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why need install?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not? it's a temporary node so doesn't hurt anybody :)