Skip to content

Commit a912b4f

Browse files
dionhaefnerclaude
andcommitted
Fix bump workflow to generate lockfile via pinned hook
The Bump UV lockfile workflow regenerated the lockfile directly with the workflow's (unpinned) uv, while the update-uv-env pre-commit hook that CI validates against pins uv==0.11.21. uv 0.11.26 changed platform-marker output syntax, so the two produced different lockfiles and the pre-commit CI check failed on bump PRs. Adopt the same approach as the internal repos: remove production.uv.lock and regenerate it via the pinned pre-commit hooks, so bot and CI use the same uv. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 1557d72 commit a912b4f

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

.github/workflows/bump_lockfile.yml

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,15 +29,20 @@ jobs:
2929
pip install toml packaging
3030
python .github/workflows/update_runtime_deps.py pyproject.toml
3131
32+
- name: Install pre-commit
33+
# uv provides the interpreter and puts pre-commit on PATH for the steps
34+
# below.
35+
run: |
36+
uv tool install pre-commit
37+
3238
- name: Update lockfile
3339
run: |
34-
mv production.uv.lock uv.lock
35-
uv lock --upgrade
36-
mv uv.lock production.uv.lock
40+
# Remove lockfile to force update
41+
rm production.uv.lock
42+
pre-commit run update-uv-env --all-files || true
3743
3844
- name: Generate new requirements.txt
3945
run: |
40-
pip install pre-commit
4146
pre-commit run update-requirements --all-files || true
4247
4348
- name: Detect if changes were made

0 commit comments

Comments
 (0)