Skip to content

Conversation

@tydeu
Copy link
Member

@tydeu tydeu commented Dec 1, 2025

This PR refactors Lake's CLI API and related code to make more use of the new String Slice API. It also generally expands the scope of the CLI API to cover argument parsing and some basic verification.

This is mostly a quality-of-life improvement for my use of the API in Lake and for adding additional CLI code in the future.

@tydeu tydeu added the changelog-no Do not include this PR in the release changelog label Dec 1, 2025
@tydeu tydeu force-pushed the lake/cli-api-overhaul branch from befe468 to 3a049c9 Compare December 1, 2025 02:32
@tydeu
Copy link
Member Author

tydeu commented Dec 1, 2025

!bench

@leanprover-radar
Copy link

leanprover-radar commented Dec 1, 2025

Benchmark results for 3a049c9 against b0e6db3 are in! @tydeu

Minor changes (3)
  • 🟥 build//instructions: +31.4G (+0.2%)
  • bv_decide_large_aig.lean//instructions: -308.9M (-0.7%)
  • 🟥 size/stdlib/.olean.server//bytes: +64kiB (+0.2%)

@tydeu tydeu force-pushed the lake/cli-api-overhaul branch from 3a049c9 to 7ca252e Compare December 1, 2025 15:30
@tydeu
Copy link
Member Author

tydeu commented Dec 1, 2025

!radar

@leanprover-radar
Copy link

leanprover-radar commented Dec 1, 2025

Benchmark results for 7ca252e against 5bd331e are in! @tydeu

Minor changes (2)
  • 🟥 build//instructions: +31.4G (+0.2%)
  • 🟥 size/all/.olean.server//bytes: +64kiB (+0.2%)

@tydeu tydeu force-pushed the lake/cli-api-overhaul branch from 7ca252e to d9d34eb Compare December 7, 2025 19:11
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Dec 7, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 7, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 7, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 7, 2025
@leanprover-bot leanprover-bot added the builds-manual CI has verified that the Lean Language Reference builds against this PR label Dec 7, 2025
@leanprover-bot
Copy link
Collaborator

leanprover-bot commented Dec 7, 2025

Reference manual CI status:

@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Dec 7, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 8, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 8, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-manual CI has verified that the Lean Language Reference builds against this PR builds-mathlib CI has verified that Mathlib builds against this PR changelog-no Do not include this PR in the release changelog toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants