-
Notifications
You must be signed in to change notification settings - Fork 111
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: fixes for new simp default (decide := false
) (lean4#2722)
#312
Merged
digama0
merged 2 commits into
leanprover-community:bump/v4.4.0
from
collares:lean-pr-testing-2722
Nov 4, 2023
Merged
chore: fixes for new simp default (decide := false
) (lean4#2722)
#312
digama0
merged 2 commits into
leanprover-community:bump/v4.4.0
from
collares:lean-pr-testing-2722
Nov 4, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
collares
changed the title
chore: fixes simp defaulting to decide := false (lean4#2722)
chore: fixes for new simp default (Oct 21, 2023
decide := false
) (lean4#2722)
collares
force-pushed
the
lean-pr-testing-2722
branch
3 times, most recently
from
October 21, 2023 20:59
a2657ee
to
60a1ddf
Compare
kim-em
added
the
depends on core changes
This PR need only be reviewed after changes land in Lean core.
label
Oct 21, 2023
kim-em
reviewed
Oct 21, 2023
These changes seem pretty reasonable to me. |
3 tasks
collares
force-pushed
the
lean-pr-testing-2722
branch
from
October 23, 2023 17:26
60a1ddf
to
ed5610b
Compare
leanprover-community-mathlib4-bot
added
the
merge-conflict
This PR has merge conflicts with the `main` branch which must be resolved by the author.
label
Oct 31, 2023
collares
force-pushed
the
lean-pr-testing-2722
branch
from
October 31, 2023 06:57
ed5610b
to
6f724a0
Compare
leanprover-community-mathlib4-bot
removed
the
merge-conflict
This PR has merge conflicts with the `main` branch which must be resolved by the author.
label
Oct 31, 2023
collares
force-pushed
the
lean-pr-testing-2722
branch
from
November 2, 2023 07:55
a81cb02
to
e930b99
Compare
awaiting-review |
github-actions
bot
added
the
awaiting-review
This PR is ready for review; the author thinks it is ready to be merged.
label
Nov 2, 2023
collares
force-pushed
the
lean-pr-testing-2722
branch
2 times, most recently
from
November 3, 2023 08:37
986f0db
to
66f20fc
Compare
@digama0, @joehendrix, could this please be reviewed and merged? Note that this will go into the After this is merged I will merge leanprover-community/mathlib4#7834 into Mathlib's |
collares
force-pushed
the
lean-pr-testing-2722
branch
from
November 3, 2023 11:27
66f20fc
to
029d701
Compare
digama0
approved these changes
Nov 3, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
awaiting-review
This PR is ready for review; the author thinks it is ready to be merged.
depends on core changes
This PR need only be reviewed after changes land in Lean core.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adapt to new default in Lean master.