GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Remove the pip-install input by v-gowridurgad · Pull Request #1327 · actions/setup-python

Conversation

@v-gowridurgad

@v-gowridurgad v-gowridurgad commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

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:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
ee-oai reacted with heart emoji
All reactions
  • ❤️ 1 reaction
gowridurgad
Remove the pip-install input
e54a887
@v-gowridurgad
v-gowridurgad requested a review from a team as a code owner June 23, 2026 08:58
Copilot AI review requested due to automatic review settings June 23, 2026 08:58
Copilot started reviewing on behalf of v-gowridurgad June 23, 2026 08:59
View session
Copilot AI reviewed Jun 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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:

  • Removes pip-install handling from the action’s TypeScript source and action.yml input schema.
  • Deletes documentation sections/links describing pip-install.
  • Drops e2e workflow jobs that exercised pip-install, and updates the compiled dist/ bundles accordingly.

Reviewed 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.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread action.yml Resolved

Uh oh!

There was an error while loading. Please reload this page.
gowridurgad
Rebuild dist
1ddad11
v-mahabaleshwars
v-mahabaleshwars approved these changes Jun 23, 2026
View reviewed changes
v-priya-kinthali
v-priya-kinthali approved these changes Jun 24, 2026
View reviewed changes
@v-gowridurgad v-gowridurgad self-assigned this Jun 24, 2026
@v-gowridurgad v-gowridurgad mentioned this pull request Jul 8, 2026
Consider removing the pip-install input #1251
Closed
@v-gowridurgad v-gowridurgad closed this by deleting the head repository Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@v-priya-kinthali v-priya-kinthali v-priya-kinthali approved these changes
@v-mahabaleshwars v-mahabaleshwars v-mahabaleshwars approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@v-gowridurgad v-gowridurgad

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@v-gowridurgad @v-priya-kinthali @v-mahabaleshwars
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.