Skip to content

Cache .repository folder and simplify use of pip cache#320

Merged
tomaskontrimas merged 1 commit into
masterfrom
ci_cache
May 22, 2026
Merged

Cache .repository folder and simplify use of pip cache#320
tomaskontrimas merged 1 commit into
masterfrom
ci_cache

Conversation

@tomaskontrimas
Copy link
Copy Markdown
Collaborator

@tomaskontrimas tomaskontrimas commented May 21, 2026

Rerunning the job seems to have restored the cache successfully:
image

Cache artifacts be found here: https://github.com/icecube/skyllh/actions/caches

The only caveat is that when we change the (expected) content of .repository we should remove the cached folder in actions/caches or increment the version key so that the cache is updated.

@tomaskontrimas tomaskontrimas self-assigned this May 21, 2026
@tomaskontrimas tomaskontrimas marked this pull request as ready for review May 22, 2026 08:36
Copilot AI review requested due to automatic review settings May 22, 2026 08:36
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 updates the GitHub Actions workflows to rely on actions/setup-python’s built-in pip caching and adds caching for the local .repository dataset folder to speed up repeated CI runs.

Changes:

  • Switch documentation workflow from a custom pip cache dir + actions/cache setup to actions/setup-python’s cache: pip.
  • Enable pip caching in the CI pytest matrix via actions/setup-python.
  • Add an actions/cache step to persist the .repository folder between CI runs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/documentation.yml Simplifies dependency caching by using setup-python’s pip cache and streamlines the install step.
.github/workflows/ci.yml Enables pip caching and introduces caching of .repository to reduce repeated dataset setup work across CI runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
Comment on lines +36 to +41
- name: Cache .repository folder
uses: actions/cache@v5
with:
path: .repository
key: repository-v1
restore-keys: repository-
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think it's fine as it happens only during the run with the first cache miss and I don't want to create another job to download to the cache first

@tomaskontrimas tomaskontrimas merged commit 40ebf8e into master May 22, 2026
5 checks passed
@tomaskontrimas tomaskontrimas deleted the ci_cache branch May 22, 2026 09:33
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