Skip to content

chore: permit Lake failures in PR release CI - #6403

Closed
tydeu wants to merge 1 commit into
leanprover:masterfrom
tydeu:pr-release-lake-failures
Closed

chore: permit Lake failures in PR release CI#6403
tydeu wants to merge 1 commit into
leanprover:masterfrom
tydeu:pr-release-lake-failures

Conversation

@tydeu

@tydeu tydeu commented Dec 16, 2024

Copy link
Copy Markdown
Member

This PR allows the PR release CI to succeed even if Lake fails to update Mathlib's version of Batteries.

This can occur if Lake fails to configure the package due to configuration file errors, such as those that may be caused by changes in the PR. This happened, for instance, in #6388.

@tydeu tydeu added the changelog-no Do not include this PR in the release changelog label Dec 16, 2024
@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 16, 2024
ghost pushed a commit to leanprover-community/batteries that referenced this pull request Dec 16, 2024
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Dec 16, 2024
@ghost ghost added the builds-mathlib CI has verified that Mathlib builds against this PR label Dec 16, 2024
@ghost

ghost commented Dec 16, 2024

Copy link
Copy Markdown

Mathlib CI status (docs):

@tydeu
tydeu marked this pull request as ready for review December 16, 2024 21:52
@tydeu
tydeu requested a review from kim-em as a code owner December 16, 2024 21:52
@kim-em

kim-em commented Jan 28, 2025

Copy link
Copy Markdown
Collaborator

I'm not sure what the purpose of this is: it's fine to merge Lean PRs that are labelled breaks-mathlib, if there's a plan to unbreak it. This just seems to hide a downstream failure from view.

@kim-em

kim-em commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

This hasn't been touched in over a year, and has a merge conflict. I'm closing now; please re-open if you think appropriate.

@kim-em kim-em closed this Sep 1, 2026
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.

2 participants