-
Notifications
You must be signed in to change notification settings - Fork 1k
[Merged by Bors] - chore(Cache): add README and rename USE_FRO_CACHE to MATHLIB_CACHE_USE_CLOUDFLARE #33647
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
[Merged by Bors] - chore(Cache): add README and rename USE_FRO_CACHE to MATHLIB_CACHE_USE_CLOUDFLARE #33647
Conversation
…E_CLOUDFLARE This PR: - Adds `Cache/README.md` documenting the cache system, environment variables, and how to set up custom endpoints - Renames `USE_FRO_CACHE` to `MATHLIB_CACHE_USE_CLOUDFLARE` for consistent `MATHLIB_CACHE_*` prefix across all cache environment variables - Updates the help text in `Cache/Main.lean` to document the `--repo` option and environment variables - Fixes help text saying "Run 'mk'" (should be "Run 'pack'") 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1ee3ccc to
266c421
Compare
PR summary 15f6691e7aImport changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diffNo declarations were harmed in the making of this PR! 🐙 You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
|
This pull request has conflicts, please merge |
Resolve conflicts by keeping master's `initialize` blocks for URL and UPLOAD_URL with MATHLIB_CACHE_GET_URL/MATHLIB_CACHE_PUT_URL support, while using the renamed `useCloudflareCache` (from `useFROCache`) and `MATHLIB_CACHE_USE_CLOUDFLARE` (from `USE_FRO_CACHE`) environment variable.
bryangingechen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
bors r+
…E_CLOUDFLARE (#33647) This PR: - Adds `Cache/README.md` documenting the cache system, environment variables, and how to set up custom endpoints - Renames `USE_FRO_CACHE` to `MATHLIB_CACHE_USE_CLOUDFLARE` for consistent `MATHLIB_CACHE_*` prefix across all cache environment variables - Updates the help text in `Cache/Main.lean` to document the `--repo` option and environment variables - Fixes help text saying "Run 'mk'" (should be "Run 'pack'") 🤖 Prepared with Claude Code
|
Pull request successfully merged into master. Build succeeded: |
…E_CLOUDFLARE (leanprover-community#33647) This PR: - Adds `Cache/README.md` documenting the cache system, environment variables, and how to set up custom endpoints - Renames `USE_FRO_CACHE` to `MATHLIB_CACHE_USE_CLOUDFLARE` for consistent `MATHLIB_CACHE_*` prefix across all cache environment variables - Updates the help text in `Cache/Main.lean` to document the `--repo` option and environment variables - Fixes help text saying "Run 'mk'" (should be "Run 'pack'") 🤖 Prepared with Claude Code
This PR:
Cache/README.mddocumenting the cache system, environment variables, and how to set up custom endpointsUSE_FRO_CACHEtoMATHLIB_CACHE_USE_CLOUDFLAREfor consistentMATHLIB_CACHE_*prefix across all cache environment variablesCache/Main.leanto document the--repooption and environment variables🤖 Prepared with Claude Code