Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix combined CI for mathlib #3700

Merged
merged 1 commit into from
Mar 17, 2024
Merged

chore: fix combined CI for mathlib #3700

merged 1 commit into from
Mar 17, 2024

Conversation

kim-em
Copy link
Collaborator

@kim-em kim-em commented Mar 16, 2024

Previously, if there was a nightly-testing-YYYY-MM-DD tag at Std, but not Mathlib, we were erroneously proceeding with Mathlib CI, and hence using a probably-broken version of Mathlib.

@kim-em kim-em requested a review from Kha as a code owner March 16, 2024 23:28
@kim-em kim-em enabled auto-merge March 16, 2024 23:28
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc March 16, 2024 23:30 Inactive
@nomeata
Copy link
Collaborator

nomeata commented Mar 16, 2024

Subsumes #3631 I presume?

@kim-em kim-em added this pull request to the merge queue Mar 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 Mar 16, 2024
@leanprover-community-mathlib4-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Std/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 0b01ceb3bb3334fd6c2f3b24996646046722c62a --onto 0ec8862103e397715854a7a6962ce542bba4884d. (2024-03-16 23:45:03)

@kim-em
Copy link
Collaborator Author

kim-em commented Mar 16, 2024

Oh, sorry, completely missed #3631.

Merged via the queue into master with commit 811bedf Mar 17, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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