Skip to content

fix(ci/cd): fix typegen test failure introduced by PR 9209#9260

Merged
lstein merged 1 commit into
mainfrom
lstein/fix/ci-failures
May 31, 2026
Merged

fix(ci/cd): fix typegen test failure introduced by PR 9209#9260
lstein merged 1 commit into
mainfrom
lstein/fix/ci-failures

Conversation

@lstein

@lstein lstein commented May 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

PR #9209 introduced a regression in the typegen check CI script manifested by the following error during the install dependencies step:

 error: No virtual environment found for Python 3.11; run `uv venv` to create an environment, or pass `--system` to install into a non-virtual environment                                           

The core issue is that after the change to setup-uv@v8 + python-version, uv 0.6.10 now refuses uv pip install
without a venv being created. This PR fixes the issue by setting the environment variable UV_SYSTEM_PYTHON which restores the earlier behavior of using the system Python.

openapi-checks has the same issue, so I fixed it here as well.

Related Issues / Discussions

QA Instructions

The PR checks should now succeed.

Merge Plan

Simple merge

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@lstein lstein requested a review from blessedcoolant as a code owner May 31, 2026 23:11
@github-actions github-actions Bot added the CI-CD Continuous integration / Continuous delivery label May 31, 2026
@lstein lstein merged commit 32b672f into main May 31, 2026
14 checks passed
@lstein lstein deleted the lstein/fix/ci-failures branch May 31, 2026 23:12
dunkeroni pushed a commit to dunkeroni/InvokeAI that referenced this pull request Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-CD Continuous integration / Continuous delivery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant