Skip to content

Pin pip tooling version in CI - #204

Merged
dtaivpp merged 1 commit into
mainfrom
chore/pin-pip-bootstrap
Oct 23, 2026
Merged

dtaivpp merged 1 commit into
mainfrom
chore/pin-pip-bootstrap

Conversation

@dtaivpp

@dtaivpp dtaivpp commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Installs pip from a hash-pinned bootstrap file (.github/pip-bootstrap.txt, pip 26.2.2, --require-hashes and --only-binary=:all:) before the first pip command in:

  • ci.yml Python bindings job (before pip install maturin)
  • release.yml wheel build job on non-Linux runners (Linux builds run inside the maturin container)
  • release.yml PyPI dry-run check, which previously used pipx run twine check: it now sets up Python, bootstraps pip, installs twine and runs python -m twine check

The wheel test jobs install with --no-index from local artifacts and make no network requests, so they are unchanged.

Verified in a python:3.13-slim container: the bootstrap file yields pip 26.2.2 and python -m twine --version runs afterwards. No image or deployment is affected; the change takes effect on the next workflow run.

Open in Capy


Summary by cubic

Pins CI pip to a hash-verified bootstrap file (.github/pip-bootstrap.txt, pip 26.2.2 with --require-hashes and --only-binary=:all:) so every pip invocation runs on a known-good version.

  • Bootstraps pip before pip install maturin in the CI Python bindings job.
  • Bootstraps pip on non-Linux release wheel build runners; Linux builds run inside the maturin container and are unaffected.
  • The PyPI dry-run check now bootstraps pip, installs twine, and runs python -m twine check instead of pipx run twine check, gaining checkout and Python setup steps for manual runs.
  • Wheel test jobs stay unchanged because they install from local artifacts with --no-index and make no network requests.

Written for commit c03ccaa. Summary will update on new commits.

View guided diff Turn on auto-fix

… on it

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

View guided diff | Turn on auto-fix | Re-trigger cubic

@capy-ai
capy-ai Bot requested a review from tomsideguide October 22, 2026 23:03
@capy-ai

capy-ai Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

No rebuild needed after merge: CI and release workflows use the pinned pip on their next run.

@capy-ai
capy-ai Bot requested review from mogery and removed request for tomsideguide October 23, 2026 07:54
@dtaivpp
dtaivpp merged commit e7178e0 into main Oct 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant