Skip to content

Commit e0cf97f

Browse files
committed
Yes — your version is much closer to the correct logic than mine, and it correctly preserves the “label is the gate” + “runs on subsequent pushes” behavior.
1 parent ba46608 commit e0cf97f

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

.github/workflows/ci.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,17 @@ jobs:
2525
name: Run on ${{ matrix.os }} with SOFA ${{ matrix.sofa_branch }} / MeshRefinement=${{ matrix.with_mesh_refinement }}
2626

2727
if: >
28-
github.event_name == 'push' ||
28+
github.event_name == 'push' || github.event_name == 'workflow_dispatch' || startsWith(github.ref, 'refs/tags/v') ||
2929
(
3030
github.event_name == 'pull_request' &&
3131
(
32-
contains(github.event.pull_request.labels.*.name, 'pr: run ci') ||
33-
github.event.action == 'labeled'
32+
contains(github.event.pull_request.labels.*.name, 'pr: run ci') &&
33+
(
34+
github.event.action != 'labeled' ||
35+
github.event.label.name == 'pr: run ci'
36+
)
3437
)
35-
) ||
36-
github.event_name == 'workflow_dispatch' ||
37-
startsWith(github.ref, 'refs/tags/v')
38+
)
3839
3940
runs-on: ${{ matrix.os }}
4041

0 commit comments

Comments
 (0)