Skip to content

fix: lake: report bad imports from a library build#13340

Merged
tydeu merged 1 commit intoleanprover:masterfrom
tydeu:lake/lib-badImport
Apr 9, 2026
Merged

fix: lake: report bad imports from a library build#13340
tydeu merged 1 commit intoleanprover:masterfrom
tydeu:lake/lib-badImport

Conversation

@tydeu
Copy link
Copy Markdown
Member

@tydeu tydeu commented Apr 9, 2026

This PR fixes a Lake issue where library builds would not produce informative errors about bad imports (unlike module builds).

@tydeu tydeu added changelog-lake Lake lake-ci Run all Lake tests labels Apr 9, 2026
@tydeu tydeu force-pushed the lake/lib-badImport branch from fc398eb to 9282d2e Compare April 9, 2026 03:18
@tydeu
Copy link
Copy Markdown
Member Author

tydeu commented Apr 9, 2026

!bench

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented Apr 9, 2026

Benchmark results for 9282d2e against 4f6bcc5 are in. There are no significant changes. @tydeu

  • build//instructions: -302.0M (-0.00%)

Small changes (1✅, 1🟥)

  • 🟥 build/module/Lake.Build.Library//instructions: +115.6M (+0.98%) (reduced significance based on *//lines)
  • elab/bv_decide_realworld//wall-clock: -50ms (-4.17%)

@tydeu tydeu marked this pull request as ready for review April 9, 2026 04:02
@tydeu tydeu added this pull request to the merge queue Apr 9, 2026
@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 Apr 9, 2026
@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2026-04-07 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2026-04-09 04:22:24)

mathlib-nightly-testing bot pushed a commit to leanprover-community/batteries that referenced this pull request Apr 9, 2026
@github-actions github-actions bot added the mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN label Apr 9, 2026
mathlib-nightly-testing bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Apr 9, 2026
Merged via the queue into leanprover:master with commit 82bb27f Apr 9, 2026
21 checks passed
@tydeu tydeu deleted the lake/lib-badImport branch April 9, 2026 05:04
@mathlib-lean-pr-testing mathlib-lean-pr-testing bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Apr 9, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

wkrozowski pushed a commit to wkrozowski/lean4 that referenced this pull request Apr 10, 2026
This PR fixes a Lake issue where library builds would not produce
informative errors about bad imports (unlike module builds).
wkrozowski pushed a commit to wkrozowski/lean4 that referenced this pull request Apr 12, 2026
This PR fixes a Lake issue where library builds would not produce
informative errors about bad imports (unlike module builds).
volodeyka pushed a commit that referenced this pull request Apr 16, 2026
This PR fixes a Lake issue where library builds would not produce
informative errors about bad imports (unlike module builds).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-lake Lake lake-ci Run all Lake tests mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN 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.

3 participants