Skip to content

Commit 72c139f

Browse files
authored
ci: Fix release-checks trigger filter (#1016)
Brings in fixes from CQCL/guppylang#1150 We were filtering the _target_ branch instead of the source, so this never ran.
1 parent cfe1689 commit 72c139f

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

.github/workflows/release-checks.yml

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22
#
33
# If this fails, it is likely that the packages require a release before we can release the `tket` python lib.
44

5-
name: tket-py Release checks
5+
name: 🚚 tket-py release checks
66

77
on:
88
pull_request:
99
branches:
10-
- release-please--branches--main--components--tket-py
10+
- main
1111
workflow_dispatch: {}
1212

1313
env:
@@ -23,6 +23,10 @@ jobs:
2323
py-release:
2424
name: Check `tket-py` release compatibility
2525
runs-on: ubuntu-latest
26+
# Check if we are running on a release PR.
27+
#
28+
# release-please always uses this branch name.
29+
if: ${{ github.ref == 'refs/heads/release-please--branches--main--components--tket-py' }}
2630
steps:
2731
- uses: actions/checkout@v4
2832
- uses: mozilla-actions/sccache-action@v0.0.9
@@ -32,11 +36,12 @@ jobs:
3236
version: ${{ env.UV_VERSION }}
3337
enable-cache: true
3438
- name: Install Python ${{ env.PYTHON_HIGHEST }}
35-
run: uv python install ${{ env.PYTHON_HIGHEST }}
39+
run: |
40+
uv python install ${{ env.PYTHON_HIGHEST }}
41+
uv python pin ${{ env.PYTHON_HIGHEST }}
3642
- name: Setup `tke2-py` with only pypi deps
3743
run: |
38-
uv sync --no-sources --no-install-workspace \
39-
--python ${{ env.PYTHON_HIGHEST }}
44+
uv sync --no-sources --no-install-workspace
4045
uv pip install --no-sources tket-exts
4146
uv pip install --no-sources tket-eccs
4247
uv run --no-sync maturin develop

0 commit comments

Comments
 (0)