Skip to content

chore(cicd): add pip cache to test workflow - #72

Closed
BobTheBuidler wants to merge 12 commits into
python:masterfrom
BobTheBuidler:you-must-comply/mypy-extensions-cache-clean-20260124123000
Closed

chore(cicd): add pip cache to test workflow#72
BobTheBuidler wants to merge 12 commits into
python:masterfrom
BobTheBuidler:you-must-comply/mypy-extensions-cache-clean-20260124123000

Conversation

@BobTheBuidler

Copy link
Copy Markdown

Summary

  • enable pip caching in the test workflow

Rationale

Caching Python dependencies reduces CI duration and speeds contributor feedback.

Details

  • actions/setup-python now uses pip cache keyed off pyproject.toml for lint and test jobs

Please forgive the unrelated ugly git history, this will still squash nicely

@cdce8p cdce8p left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think adding caching makes sense here. mypy_extensions doesn't have any dependencies, not even for tests. (Unless I'm missing something.) Furthermore the job runtime was just ~30s each.

@BobTheBuidler

Copy link
Copy Markdown
Author

Makes sense, just kinda noticed it wasn't using caching while I was poking around, but I never spent any time looking into the deps.

Closing this.

@BobTheBuidler
BobTheBuidler deleted the you-must-comply/mypy-extensions-cache-clean-20260124123000 branch February 8, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants