Skip to content

Commit 64ffbee

Browse files
committed
CI: skip jep tests when running the javaless job
* Include java version in job names, to disambiguate them. * As a hack for now, just fail if Java >= v17 or <8.
1 parent ace4df5 commit 64ffbee

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ on:
1212

1313
jobs:
1414
build-cross-platform:
15-
name: test ${{matrix.os}} - ${{matrix.python-version}}
15+
name: test ${{matrix.os}} - ${{matrix.python-version}} - ${{matrix.java-version}}
1616
runs-on: ${{ matrix.os }}
1717
strategy:
1818
matrix:
@@ -33,7 +33,6 @@ jobs:
3333
python-version: '3.12'
3434
java-version: ''
3535

36-
3736
steps:
3837
- uses: actions/checkout@v2
3938

bin/test.sh

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@ then
7373
else
7474
argString=""
7575
fi
76-
if [ "$(uname -s)" = "Darwin" ]
76+
if ! java -version 2>&1 | grep -q '^openjdk version "\(1\.8\|9\|10\|11\|12\|13\|14\|15\|16\)\.'
77+
then
78+
echo "Skipping jep tests due to unsupported Java version:"
79+
java -version || true
80+
jepCode=0
81+
elif [ "$(uname -s)" = "Darwin" ]
7782
then
7883
echo "Skipping jep tests on macOS due to flakiness"
7984
jepCode=0

0 commit comments

Comments
 (0)