-
Notifications
You must be signed in to change notification settings - Fork 121
New big_uncond
and big_rmcond -> big_rmcond_in
#492
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
Conversation
65c4f9b
to
3da363c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Lemma `big_rmcond` has been renamed `big_rmcomd_in` and the variant which does not require an `eqType` has been added and named `big_uncond`. - The name `big_rmcond` is deprecated and will be reused for `big_uncond` in the next version of math-comp - Additionally `big_uncond_in` is made available for uniformity, but is already deprecated.
3da363c
to
1107e77
Compare
big_uncond
and big_rmcond -> big_rmcond_in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@CohenCyril Could you take care of the remaining TODO? If you are happy with the current status, I will merge. |
It's a todo "after merge" ;) |
once you merge, I do it |
@CohenCyril Thanks! |
Motivation for this change
big_rmcond
has been renamedbig_rmcomd_in
and the variant which does not require aneqType
has been added and namedbig_uncond
.big_rmcond
is deprecated and will be reused forbig_uncond
in the next version of math-compbig_uncond_in
is made available for uniformity, but is already deprecated.Things done/to do
CHANGELOG_UNRELEASED.md
added corresponding documentation in the headersAutomatic note to reviewers
Read this Checklist and make sure there is a milestone.