Skip to content

perf: restore .lake before configuration and mk_all - #199

Open
SamuelSchlesinger wants to merge 1 commit into
leanprover:mainfrom
SamuelSchlesinger:samschles/restore-cache-earlier
Open

SamuelSchlesinger wants to merge 1 commit into
leanprover:mainfrom
SamuelSchlesinger:samschles/restore-cache-earlier

Conversation

@SamuelSchlesinger

Copy link
Copy Markdown

Restore the GitHub .lake cache before automatic configuration and mk_all-check, allowing both steps to reuse cached dependencies and build artifacts. Cache keys, conditions, and the relative order of all other steps remain unchanged.

In three local trials using Mathlib’s mk_all on a small downstream package, median runtime fell from 20.1s to 6.45s, with rebuilt targets dropping from 137 to zero. These measurements exclude cache transfer and dependency downloads.

Validation passed ShellCheck, actionlint, and YAML parsing. Restored caches correctly detected changed configuration and newly unimported files.

Codex assisted with implementation and validation.

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.

1 participant