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

fix: remove simp from compare_self #7218

Closed
wants to merge 2 commits into from

Conversation

JovanGerb
Copy link
Contributor

This PR removes the simp tag from compare_self, because simp tries this lemma at every subexpression.

@JovanGerb JovanGerb requested a review from TwoFX as a code owner February 25, 2025 00:00
@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 Feb 25, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Feb 25, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Feb 25, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Feb 25, 2025
@TwoFX TwoFX added the changelog-library Library label Feb 25, 2025
@TwoFX TwoFX added this pull request to the merge queue Feb 25, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 25, 2025
@datokrat
Copy link
Contributor

datokrat commented Feb 25, 2025

This PR will be superseded by #7222, which also resolves some errors with more recent changes. @JovanGerb feel free to let me know if you would like to change something! (And thanks for proposing this very helpful change -- me creating a new PR doesn't mean that anything was wrong with this one.)

@TwoFX TwoFX closed this Feb 25, 2025
@JovanGerb JovanGerb deleted the unsimp-compare_self branch February 25, 2025 13:41
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-library Library 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.

4 participants