Skip to content

UV in Compile Script#3

Merged
nickchecan merged 1 commit intomainfrom
fix-compile
Apr 9, 2026
Merged

UV in Compile Script#3
nickchecan merged 1 commit intomainfrom
fix-compile

Conversation

@nickchecan
Copy link
Copy Markdown
Owner

This pull request refactors the compile_test.sh unit test to focus on the new uv-based Python dependency management workflow, removes legacy pip bootstrapping logic, and updates the test stubs and assertions to match the new approach. The most important changes are grouped below.

Test logic and coverage updates:

  • Removes the test_compile_bootstraps_pip_when_missing test and all related code, reflecting the move away from pip bootstrapping in favor of uv for dependency management. [1] [2]

  • Updates and expands assertions in test_compile_succeeds_for_locked_uv_project to check for Python version installation from .python-version, creation of Python shims, correct updates to the profile script, and uv-based dependency installation steps. [1] [2]

Test environment and stub refactoring:

  • Refactors the fake command setup to use a managed Python shim (managed-python3) and updates the uv stub to handle new subcommands (python install, python find, pip install, and export). Removes the old pip and ensurepip stubs. [1] [2] [3] [4]

  • Ensures the .python-version file is created in relevant tests to simulate Python version pinning. [1] [2] [3]

Test runner environment:

  • Updates run_compile to pass the FAKE_MANAGED_PYTHON environment variable instead of FAKE_PIP_AVAILABLE.

@nickchecan nickchecan merged commit 1e976e3 into main Apr 9, 2026
3 checks passed
@nickchecan nickchecan deleted the fix-compile branch April 9, 2026 16:09
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🎉 This PR is included in version 1.0.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant