Skip to content

Fix: initialize writable pip cache directory - #277

Merged
rapids-bot[bot] merged 1 commit into
mainfrom
codex/pip-cache-dir-prototype
Sep 9, 2026
Merged

rapids-bot[bot] merged 1 commit into
mainfrom
codex/pip-cache-dir-prototype

Conversation

@vyasr

@vyasr vyasr commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Description

Set PIP_CACHE_DIR to a writable, workspace-local directory when callers do not supply one, while preserving an explicit caller value. This enables a reusable pip cache in GitHub Actions job containers.

Refs rapidsai/build-planning#260.

Validation

  • pre-commit run --files tools/rapids-init-pip
  • Verified both the default workspace path and an explicit caller-supplied path in an isolated shell.

@vyasr vyasr mentioned this pull request Sep 9, 2026
2 of 3 tasks
@vyasr
vyasr marked this pull request as ready for review September 9, 2026 21:50
@vyasr
vyasr requested a review from a team as a code owner September 9, 2026 21:50
@vyasr
vyasr requested review from msarahan and removed request for a team September 9, 2026 21:50

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yay 🎉

@vyasr

vyasr commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit 495f0eb into main Sep 9, 2026
3 checks passed
@vyasr vyasr mentioned this pull request Sep 9, 2026
2 of 3 tasks
@vyasr
vyasr deleted the codex/pip-cache-dir-prototype branch September 9, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants