Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adapt azure cleanup script to allow caches from non-mathlib repos #9

Merged
merged 4 commits into from
Oct 3, 2021

Conversation

robertylewis
Copy link
Member

A companion to leanprover-community/lean-liquid#48

This allows non-mathlib leanprover-community repos to store caches on Azure. The deletion conditions are tailored for lean-liquid right now: only master commits are kept and only for three days. This is easy enough to change.

@bryangingechen
Copy link
Contributor

I completely missed this. Right now lean-liquid caches are being uploaded but not deleted, and this PR would clean up the old ones, is that right?

@robertylewis
Copy link
Member Author

I probably forgot about this by July 1, but yeah, I think you're right. Or maybe right now lean-liquid caches are getting deleted as soon as the script runs?

@bryangingechen
Copy link
Contributor

Right, I just checked the most recent log and indeed it looks like all lean-liquid caches are being deleted.

LGTM.

.github/workflows/cleanup.yml Outdated Show resolved Hide resolved
@robertylewis robertylewis merged commit 867c5e6 into master Oct 3, 2021
@robertylewis robertylewis deleted the non-mathlib-cleanup branch October 3, 2021 19:01
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.

None yet

2 participants