Skip to content

Comments

Add WSL testing for pyenv-win#15317

Merged
zanieb merged 1 commit intomainfrom
zb/wsl-test
Aug 18, 2025
Merged

Add WSL testing for pyenv-win#15317
zanieb merged 1 commit intomainfrom
zb/wsl-test

Conversation

@zanieb
Copy link
Member

@zanieb zanieb commented Aug 15, 2025

Adds test coverage for #15315

I reproduced the error there, then rebased on #15315 which resolves the issue.

@zanieb zanieb changed the title zb/wsl test Add WSL testing Aug 15, 2025
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 20:47 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 20:47 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 21:02 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 21:02 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 21:13 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 21:18 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 21:18 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 22:25 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 22:25 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 22:33 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 22:33 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 22:41 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 22:41 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/wsl-test branch 2 times, most recently from a353c14 to b79ab9e Compare August 15, 2025 22:46
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 22:48 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 15, 2025 22:55 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 15, 2025 22:55 — with GitHub Actions Inactive
@zanieb zanieb changed the base branch from main to zb/not-found-on-query August 15, 2025 22:55
@zanieb zanieb changed the title Add WSL testing Add WSL testing for pyenv-win Aug 15, 2025
@zanieb zanieb marked this pull request as ready for review August 15, 2025 22:55
@zanieb zanieb added the testing Internal testing of behavior label Aug 15, 2025
zanieb added a commit that referenced this pull request Aug 18, 2025
Closes #12155

We already throw this error earlier if we cannot find the interpreter
https://github.com/astral-sh/uv/blob/c318e8860e69077fdfe8fff7356985e684591b40/crates/uv-python/src/interpreter.rs#L1039

Why the pyenv-win shim _exists_ but fails to run with a not found error
is beyond me. I think I'll take the incremental improvement here by just
ignoring it. We can try to support their shims later?

#15317 confirms the fix.
Base automatically changed from zb/not-found-on-query to main August 18, 2025 15:42
@zanieb zanieb temporarily deployed to uv-test-registries August 18, 2025 15:46 — with GitHub Actions Inactive
@zanieb zanieb merged commit ceacd27 into main Aug 18, 2025
95 of 96 checks passed
@zanieb zanieb deleted the zb/wsl-test branch August 18, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Internal testing of behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant