Skip to content

fix: aborting elaboration should not lead to empty #print axioms #8214

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kha
Copy link
Member

@Kha Kha commented May 3, 2025

This PR fixes a UI issue where a previous error masks the result of #print axioms.

Because an error has already happened and the file would be rejected either way, this is not a consistency issue.

@Kha Kha added awaiting-mathlib We should not merge this until we have a successful Mathlib build changelog-other labels May 3, 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 May 3, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request May 3, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request May 3, 2025
@leanprover-community-bot leanprover-community-bot removed the awaiting-mathlib We should not merge this until we have a successful Mathlib build label May 3, 2025
@leanprover-community-bot
Copy link
Collaborator

@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label May 3, 2025
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-other 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