Skip to content

Robin aws/lake skimmer - #1061

Draft
robin-aws wants to merge 23 commits into
mainfrom
robin-aws/lake-skimmer
Draft

Robin aws/lake skimmer#1061
robin-aws wants to merge 23 commits into
mainfrom
robin-aws/lake-skimmer

Conversation

@robin-aws

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Apr 28, 2026
Comment thread docs/LakeCacheSkimmer.md
Comment on lines +46 to +47
- **`declaration`** (`def`, `theorem`, etc.) is pure.
- **Inspection commands** (`#check`, `#print`, etc.) are pure.

@tydeu tydeu May 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

FYI, any command that elaborates an arbitrary Lean term (e.g., def, theorem, #check) is possibly impure. A term can execute arbitrary code during elaboration through both custom elaborators and builtin ones (e.g., run_tac, include_str).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah thanks for the heads up. This "audit" was entirely Kiro-driven and I hadn't even reviewed it myself yet. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file Git conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants