Skip to content

Conversation

@tydeu
Copy link
Member

@tydeu tydeu commented Nov 12, 2025

This PR adds a test replicating Kim's diamond dependency example.

The top-level package, D, depends on two intermediate packages, B and C, which each require semantically different versions of another package, A. The portion of A that B and C publicly use is unchanged across the versions, but they both privately make use of changed API. Currently, this causes a version clash. This will be made to work without error later this quarter.

@tydeu tydeu added the changelog-no Do not include this PR in the release changelog label Nov 12, 2025
@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 Nov 12, 2025
@leanprover-bot
Copy link
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2025-11-02 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. (2025-11-12 23:29:13)

leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Nov 12, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Nov 12, 2025
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Nov 13, 2025
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Nov 13, 2025

Mathlib CI status (docs):

leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Nov 13, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Nov 13, 2025
@tydeu tydeu marked this pull request as ready for review November 13, 2025 05:40
@tydeu tydeu enabled auto-merge November 13, 2025 05:40
@tydeu tydeu added this pull request to the merge queue Nov 13, 2025
Merged via the queue into leanprover:master with commit 2b85e29 Nov 13, 2025
18 checks passed
@tydeu tydeu deleted the ver-clash-test branch November 13, 2025 07:28
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-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.

3 participants