Skip to content

Commit 67b1e65

Browse files
committed
change per review
1 parent 1abbcc6 commit 67b1e65

File tree

2 files changed

+21
-24
lines changed

2 files changed

+21
-24
lines changed

.github/workflows/release.yml

Lines changed: 13 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ on:
1717
# - master
1818

1919
jobs:
20-
# tests and tests-no-docker are from tests.yml and build the actual native libraries that will be used in the release.
21-
tests:
22-
name: Java ${{ matrix.java-version }} ${{ matrix.os }} ${{ matrix.dockcross-only }}
20+
# Corresponsd to tests in tests.yml
21+
build-with-docker:
22+
name: Build ${{ matrix.dockcross-only }} (Dockcross)
2323
runs-on: ${{ matrix.os }}
2424

2525
strategy:
@@ -66,35 +66,29 @@ jobs:
6666
restore-keys: |
6767
${{ runner.os }}-gradle-
6868
69-
# - name: Format check
70-
# if: ${{ matrix.java-version != 8 }}
71-
# run: ./gradlew spotlessCheck
72-
7369
- name: Tests
7470
run: ./gradlew clean test -Ph3SystemPrune=true "-Ph3DockcrossOnly=${{ matrix.dockcross-only }}"
7571
env:
7672
OCI_EXE: docker
7773

78-
- name: Format check for C
79-
run: git diff --exit-code
80-
8174
- uses: actions/upload-artifact@v4
8275
name: Upload artifacts
83-
if: ${{ matrix.java-version == 8 }}
8476
with:
8577
name: docker-built-shared-objects-${{ matrix.dockcross-only }}
8678
path: |
8779
src/main/resources/*/*.so
8880
src/main/resources/*/*.dll
8981
if-no-files-found: error
9082

91-
tests-no-docker:
92-
name: Java (No Docker) ${{ matrix.java-version }} ${{ matrix.os }}
83+
# Corresponsd to tests-no-docker in tests.yml
84+
build:
85+
name: Build ${{ matrix.os }}
9386
runs-on: ${{ matrix.os }}
9487

9588
strategy:
9689
matrix:
97-
os: [macos-latest]
90+
# macos-13 is for Intel Macs
91+
os: [macos-latest, macos-13]
9892
java-distribution: [adopt]
9993
java-version: [8]
10094

@@ -127,9 +121,8 @@ jobs:
127121

128122
- uses: actions/upload-artifact@v4
129123
name: Upload Mac OS Artifacts
130-
if: ${{ matrix.os == 'macos-latest' && matrix.java-version == 8 }}
131124
with:
132-
name: macos-built-shared-objects
125+
name: macos-built-shared-objects-${{ matrix.os }}
133126
path: src/main/resources/*/*.dylib
134127
if-no-files-found: error
135128

@@ -139,8 +132,8 @@ jobs:
139132
contents: write # allow pushing commits/tags
140133

141134
needs:
142-
- tests
143-
- tests-no-docker
135+
- build-with-docker
136+
- build
144137

145138
steps:
146139
- name: Check out code
@@ -189,7 +182,8 @@ jobs:
189182
- name: Download Mac binaries
190183
uses: actions/[email protected]
191184
with:
192-
name: macos-built-shared-objects
185+
name: macos-built-shared-objects-*
186+
merge-multiple: true
193187
path: src/main/resources/
194188

195189
- name: Download and test

.github/workflows/tests.yml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -114,9 +114,10 @@ jobs:
114114

115115
strategy:
116116
matrix:
117+
# macos-13 is for Intel Macs
117118
# TODO: Docker on macos-latest running is not working
118119
# TODO: Windows pinned back
119-
os: [macos-latest, windows-2019]
120+
os: [macos-latest, macos-13, windows-2019]
120121
java-distribution: [adopt]
121122
java-version: [8, 11, 17, 21, 22]
122123

@@ -149,9 +150,9 @@ jobs:
149150

150151
- uses: actions/upload-artifact@v4
151152
name: Upload Mac OS Artifacts
152-
if: ${{ matrix.os == 'macos-latest' && matrix.java-version == 8 }}
153+
if: ${{ (matrix.os == 'macos-latest' || matrix.os == 'macos-13') && matrix.java-version == 8 }}
153154
with:
154-
name: macos-built-shared-objects
155+
name: macos-built-shared-objects-${{ matrix.os }}
155156
path: src/main/resources/*/*.dylib
156157
if-no-files-found: error
157158

@@ -242,7 +243,8 @@ jobs:
242243
- name: Download Mac binaries
243244
uses: actions/[email protected]
244245
with:
245-
name: macos-built-shared-objects
246+
name: macos-built-shared-objects-*
247+
merge-multiple: true
246248
path: src/main/resources/
247249

248250
- name: Download and test
@@ -298,7 +300,8 @@ jobs:
298300
- name: Download Mac binaries
299301
uses: actions/[email protected]
300302
with:
301-
name: macos-built-shared-objects
303+
name: macos-built-shared-objects-*
304+
merge-multiple: true
302305
path: src/main/resources/
303306

304307
- name: Download and test

0 commit comments

Comments
 (0)