Skip to content

Commit a565f21

Browse files
committed
ci: don't run for luajit-2.0 on macOS because architecture is not supported
1 parent 6f20804 commit a565f21

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

.github/workflows/check.yml

+11-5
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,19 @@ jobs:
101101
needs: lint
102102

103103
strategy:
104+
fail-fast: false
104105
matrix:
105106
os: [ ubuntu-22.04, windows-latest, macos-latest ]
106107
lua_version: [ '5.1', '5.2', '5.3', '5.4', 'luajit-2.0', 'luajit-2.1' ]
108+
exclude:
109+
# Currently does not work for LuaJIT on Windows (Lua library not found)
110+
- os: windows-latest
111+
lua_version: 'luajit-2.0'
112+
- os: windows-latest
113+
lua_version: 'luajit-2.1'
114+
# Currently does not work for LuaJIT 2.0 on macOS (Architecture not supported)
115+
- os: macos-latest
116+
lua_version: 'luajit-2.0'
107117
runs-on: ${{ matrix.os }}
108118

109119
steps:
@@ -125,17 +135,13 @@ jobs:
125135

126136
- name: Build fenster
127137
run: luarocks make
128-
# Currently does not work for LuaJIT on Windows (lua library not found)
129-
if: matrix.os != 'windows-latest' || matrix.lua_version != 'luajit'
130138

131139
- name: Run tests
132140
run: luarocks test -- --exclude-tags needsdisplay --verbose --output TAP
133-
# Currently does not work for LuaJIT on Windows (see above)
134-
if: matrix.os != 'windows-latest' || matrix.lua_version != 'luajit'
135141

136142

137143
test-docker:
138-
name: Test (Docker)
144+
name: Test (docker)
139145
needs: lint
140146

141147
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)