Skip to content

Conversation

@tydeu
Copy link
Member

@tydeu tydeu commented Dec 3, 2025

This PR adds a workspace-index to the name of the package used by build target. To clarify the distinction between the different uses of a package's name, this PR also deprecates Package.name for more use-specific variants (e.g., Package.keyName, Package.prettyName, Package.origName).

More to come. (WIP)

@tydeu tydeu added the changelog-lake Lake label Dec 3, 2025
@tydeu tydeu changed the title refactor: disambiguate packages by workspace index refactor: lake: disambiguate packages by workspace index Dec 3, 2025
@tydeu tydeu force-pushed the lake/pkg-name-refactor branch from 847503f to cf9c131 Compare December 4, 2025 17:22
@tydeu tydeu force-pushed the lake/pkg-name-refactor branch from cf9c131 to 8d734cb Compare December 4, 2025 17:53
@tydeu
Copy link
Member Author

tydeu commented Dec 4, 2025

!bench

@leanprover-radar
Copy link

leanprover-radar commented Dec 4, 2025

Benchmark results for 99c0d27 against dd28f00 are in! @tydeu

@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 4, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 4, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 4, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 4, 2025
@leanprover-bot leanprover-bot added the breaks-manual This is not necessarily a blocker for merging, but there needs to be a plan. label Dec 4, 2025
@leanprover-bot
Copy link
Collaborator

leanprover-bot commented Dec 4, 2025

Reference manual CI status:

@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Dec 4, 2025

Mathlib CI status (docs):

@tydeu tydeu force-pushed the lake/pkg-name-refactor branch from 34c38e9 to 7f7915f Compare December 5, 2025 19:26
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 5, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 5, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 5, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 6, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 6, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 6, 2025
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Dec 6, 2025
@tydeu
Copy link
Member Author

tydeu commented Dec 8, 2025

!bench

@leanprover-radar
Copy link

leanprover-radar commented Dec 8, 2025

Benchmark results for 3200264 against dd28f00 are in! @tydeu

@tydeu tydeu marked this pull request as ready for review December 8, 2025 23:53
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Dec 9, 2025
@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 Dec 9, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Dec 9, 2025
leanprover-bot added a commit to leanprover/reference-manual that referenced this pull request Dec 9, 2025
@tydeu tydeu added this pull request to the merge queue Dec 9, 2025
Merged via the queue into leanprover:master with commit 1d0d391 Dec 9, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaks-manual This is not necessarily a blocker for merging, but there needs to be a plan. builds-mathlib CI has verified that Mathlib builds against this PR changelog-lake Lake 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.

4 participants