Skip to content

Make venv CI independent of the runner Conda environment - #26290

Merged
Eduardo Villalpando Mello (edvilme) merged 2 commits into
mainfrom
copilot/fix-conda-environment-reliance
Nov 13, 2026
Merged

Eduardo Villalpando Mello (edvilme) merged 2 commits into
mainfrom
copilot/fix-conda-environment-reliance

Conversation

Copilot AI commented Nov 12, 2026 •

Copy link
Copy Markdown

Venv CI jobs assumed the hosted runner exposed $Env:CONDA, causing setup to fail before tests when the variable was unset.

  • Conda setup

    • Install Conda with a commit-pinned setup-miniconda action.
    • Preserve the matrix-selected Python by disabling environment activation.
  • Path discovery

    • Resolve the first Conda executable on PATH.
    • Derive the base installation with conda info --base.
    • Record valid condaExecPath and condaPath values in envPaths.json.
  • Diagnostics

    • Report explicit errors when Conda or its base Python cannot be found.
$condaCommand = Get-Command conda -CommandType Application |
    Select-Object -First 1
$condaBasePath = & $condaCommand.Source info --base

Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI venv jobs to not rely on hosted runner's CONDA variable Make venv CI independent of the runner Conda environment Nov 13, 2026
@edvilme Eduardo Villalpando Mello (edvilme) added the debt Code quality issues label Nov 13, 2026
@edvilme
Eduardo Villalpando Mello (edvilme) marked this pull request as ready for review November 13, 2026 00:39
@rchiodo

Copy link
Copy Markdown

🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR.

@rchiodo

Copy link
Copy Markdown

Reviewed commit: 335f6516a5aa3935a129d1187a8247fc0fa618f5

Result: 🔴 could-not-verify

Verification details

Verification: Automated verification is best effort. The PR author remains responsible for testing.

Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine &#x27;pyrx-automation&#x27; is not running.

Test runs: none recorded.

@rchiodo Rich Chiodo (rchiodo) 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.

Approved via Review Center.

@rchiodo Rich Chiodo (rchiodo) added the review-auto:approved Automated review: no blocking findings (approval posted). label Nov 13, 2026
@edvilme
Eduardo Villalpando Mello (edvilme) merged commit b5f6600 into main Nov 13, 2026
99 of 100 checks passed
@edvilme
Eduardo Villalpando Mello (edvilme) deleted the copilot/fix-conda-environment-reliance branch November 13, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Code quality issues review-auto:approved Automated review: no blocking findings (approval posted).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI venv jobs should not rely on the hosted runner's CONDA environment variable

4 participants