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.

Sign up for GitHub

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

Jump to bottom

chore: fix test exclusion #5990

Merged
merged 1 commit into from
Nov 7, 2024
Merged

chore: fix test exclusion #5990

merged 1 commit into from
Nov 7, 2024

Conversation

Kha
Copy link
Member

@Kha Kha commented Nov 7, 2024

You cannot pass -E to ctest multiple times

@Kha
chore: fix test exclusion
9ebd714 
You cannot pass `-E` to `ctest` multiple times
@Kha Kha requested a review from kim-em as a code owner November 7, 2024 09:42
@Kha Kha enabled auto-merge November 7, 2024 09:42
@Kha Kha added the release-ci Enable all CI checks for a PR, like is done for releases label Nov 7, 2024
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc November 7, 2024 09:48 Inactive
@Kha Kha added this pull request to the merge queue Nov 7, 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 Nov 7, 2024
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 59ee47ad44e46b50f6c73567622476097b22ae9c --onto c779f3a039963fd38b03a78f635f0a7c36f24f42. (2024-11-07 10:45:52)

Merged via the queue into master with commit 5f7a40a Nov 7, 2024
25 checks passed
@Kha Kha deleted the fix- branch November 7, 2024 12:04
@kim-em kim-em added the changelog-no Do not include this PR in the release changelog label Jan 4, 2025
JovanGerb pushed a commit to JovanGerb/lean4 that referenced this pull request Jan 21, 2025
@Kha @JovanGerb
chore: fix test exclusion (leanprover#5990)
094e264 
You cannot pass `-E` to `ctest` multiple times
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kim-em kim-em Awaiting requested review from kim-em kim-em is a code owner

Assignees
No one assigned
Labels
changelog-no Do not include this PR in the release changelog release-ci Enable all CI checks for a PR, like is done for releases toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Kha @leanprover-community-bot @kim-em