Skip to content

Commit 5d12245

Browse files
committed
Fix conditionals in CI script
1 parent 62464f6 commit 5d12245

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

.github/workflows/main.yml

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ jobs:
3737
- name: Install OTP and Elixir
3838
uses: erlef/setup-beam@v1
3939
with:
40-
otp-version: ${{matrix.erlang}}
41-
elixir-version: ${{matrix.elixir}}
40+
otp-version: ${{ matrix.erlang }}
41+
elixir-version: ${{ matrix.elixir }}
4242

4343
- name: Cache dependencies
4444
id: cache-deps
@@ -60,15 +60,16 @@ jobs:
6060

6161
- name: Check for unused dependencies
6262
run: mix deps.get && mix deps.unlock --check-unused
63-
if: ${{ matrix.lint && steps.cache-deps.outputs.cache-hit != 'true' }}
63+
if: matrix.lint && steps.cache-deps.outputs.cache-hit != 'true'
6464

6565
- name: Compile with --warnings-as-errors
6666
run: mix compile --warnings-as-errors
67-
if: ${{ matrix.lint }}
67+
if: matrix.lint
6868

6969
- name: Restore cached PLTs
7070
uses: actions/cache@v4
7171
id: plt_cache
72+
if: ${{ matrix.dialyzer }}
7273
with:
7374
key: |
7475
${{ runner.os }}-${{ matrix.elixir }}-${{ matrix.erlang }}-plt
@@ -79,24 +80,24 @@ jobs:
7980
8081
# Create PLTs if no cached PLTs were found
8182
- name: Create PLTs
82-
if: steps.plt_cache.outputs.cache-hit != 'true'
83+
if: matrix.dialyzer && steps.plt_cache.outputs.cache-hit != 'true'
8384
run: MIX_ENV=test mix dialyzer --plt
8485

8586
- name: Run dialyzer
8687
run: MIX_ENV=test mix dialyzer
87-
if: ${{ matrix.dialyzer }}
88+
if: matrix.dialyzer
8889

8990
- name: Run tests with coverage
9091
run: mix coveralls.github
91-
if: ${{ matrix.coverage }}
92+
if: matrix.coverage
9293

9394
- name: Run tests
9495
run: mix test --trace
95-
if: ${{ !matrix.coverage }}
96+
if: '!matrix.coverage'
9697

9798
- name: Check mix format
9899
run: mix format --check-formatted
99-
if: ${{ matrix.lint }}
100+
if: matrix.lint
100101

101102
- name: Add seedling favicon to autobahn report
102103
if: github.ref == 'refs/heads/main' && matrix.report

0 commit comments

Comments
 (0)