8000 New `big_uncond` and `big_rmcond -> big_rmcond_in` by CohenCyril · Pull Request #492 · math-comp/math-comp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 1 commit into from
Sep 9, 2020

Conversation

CohenCyril
Copy link
Member
@CohenCyril CohenCyril commented May 1, 2020
Motivation for this change
  • 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.
Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • after merge: open an issue to deal with the deprecations in 1.13 and 1.14.
  • added corresponding documentation in the headers
Automatic note to reviewers

Read this Checklist and make sure there is a milestone.

@CohenCyril CohenCyril added this to the 1.12.0 milestone May 1, 2020
@CohenCyril CohenCyril requested review from pi8027 and chdoc and removed request for pi8027 September 3, 2020 01:06
Copy link
Member
@chdoc chdoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

8000
- 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.
@CohenCyril CohenCyril changed the title big_rmcond -> big_rmcond_in and big_rmcond reused New big_uncond and big_rmcond -> big_rmcond_in Sep 3, 2020
Copy link
Member
@pi8027 pi8027 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pi8027 pi8027 self-assigned this Sep 9, 2020
@pi8027
Copy link
Member
pi8027 commented Sep 9, 2020

@CohenCyril Could you take care of the remaining TODO? If you are happy with the current status, I will merge.

@CohenCyril
Copy link
Member Author

@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" ;)

@CohenCyril
Copy link
Member Author

It's a todo "after merge" ;)

once you merge, I do it

@pi8027 pi8027 merged commit c2e7bad into math-comp:master Sep 9, 2020
@pi8027
Copy link
Member
pi8027 commented Sep 9, 2020

@CohenCyril Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0