Skip to content

fix(maintainer): drop pip cache from setup-python#65

Merged
ianlintner merged 1 commit into
mainfrom
fix/maintainer-drop-pip-cache
Apr 28, 2026
Merged

fix(maintainer): drop pip cache from setup-python#65
ianlintner merged 1 commit into
mainfrom
fix/maintainer-drop-pip-cache

Conversation

@ianlintner
Copy link
Copy Markdown
Owner

The thin streaming Caretaker workflow has no actions/checkout step, so actions/setup-python@v5 with cache: "pip" errors out before pip ever runs:

##[error]No file in /home/runner/work/<repo>/<repo> matched to [**/requirements.txt or **/pyproject.toml], make sure you have checked out the target repository

Drop the line. The thin client installs exactly one wheel (caretaker) per scheduled run; pip caching adds negligible value, and adding a checkout / synthetic dependency file purely to enable cache would be more code for no benefit.

The upstream template was fixed in ianlintner/caretaker#636. This PR ports the same one-line change to the deployed copy in this repo, since caretaker's bootstrap agent only writes templates on initial install.

🤖 Generated with Claude Code

@ianlintner ianlintner merged commit 461ec34 into main Apr 28, 2026
8 checks passed
@ianlintner ianlintner deleted the fix/maintainer-drop-pip-cache branch April 28, 2026 01:38
@care-taker-bot
Copy link
Copy Markdown

@claude caretaker is requesting a full code review for this PR.

Repo: ianlintner/python_dsa · PR: #65
Routing reason: score=23 [sensitive_files(+23)] → inline

Please review this pull request for:

  • Correctness and logic errors
  • Security vulnerabilities or unsafe patterns
  • API contract and backward-compatibility concerns
  • Test coverage gaps
  • Any blocking issues before merge

To have your review surface in the GitHub Reviews tab (not just an issue comment), end your reply with the marker line <!-- caretaker:review-result --> followed by a fenced JSON block tagged caretaker-review. Example:

<!-- caretaker:review-result -->
```caretaker-review
{
  "verdict": "COMMENT",          // APPROVE | COMMENT | REQUEST_CHANGES
  "summary": "1-3 sentence overall assessment.",
  "comments": [                  // optional, max 8
    {"path": "src/foo.py", "line": 42, "body": "..."}
  ]
}
```

Caretaker will pick the JSON up on its next cycle and post a formal PR review (with inline comments) attributed to the-care-taker[bot] so it counts in the Reviews tab. If you don't include the marker, your review still posts as a regular comment, just outside the Reviews tab.

Delegated by caretaker's PRReviewerAgent via anthropics/claude-code-action hand-off.

@care-taker-bot care-taker-bot Bot added the claude-code claude-code-action trigger label Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-code claude-code-action trigger

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant