| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
This PR removes the recently added pip-install input from actions/setup-python, including its runtime implementation, documentation references, and related e2e workflow coverage.
Changes:
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file| File | Description |
|---|---|
| src/setup-python.ts | Removes the pip-install execution path and the associated @actions/exec usage. |
| action.yml | Deletes the pip-install input from the action interface. |
| README.md | Removes the pip-install link from the README table of contents. |
| docs/advanced-usage.md | Removes the pip-install documentation section and TOC entry. |
| dist/setup/index.js | Updates the generated bundle to remove pip-install logic (and includes additional large bundle churn). |
| dist/cache-save/index.js | Updates the generated post bundle (also includes additional large bundle churn). |
| .github/workflows/e2e-cache.yml | Removes pip-install-specific e2e cache jobs. |
| .github/workflows/e2e-cache-freethreaded.yml | Removes pip-install-specific e2e cache jobs for freethreaded builds. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Description:
This PR removes the pip-install input added in v6.1.0 (#1201). The input was meant to make installing dependencies easier, but in practice it installs packages into the global Python environment on the runner, which can cause unexpected behaviour with virtual environments and tools like uv, and the inline dependencies aren't picked up by tools like Dependabot.
Related issue:
#1251
Check list: