Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - chore(data/nat/basic): split out even_odd_rec#17759

Closed
kim-em wants to merge 2 commits intomasterfrom
even_odd_rec
Closed

[Merged by Bors] - chore(data/nat/basic): split out even_odd_rec#17759
kim-em wants to merge 2 commits intomasterfrom
even_odd_rec

Conversation

@kim-em
Copy link
Copy Markdown
Collaborator

@kim-em kim-em commented Nov 29, 2022

even_odd_rec is annoying to port, because it is thoroughly tangled with bit0 and bit1. It isn't used in mathlib, so I'm dumping it into its own file so we can get on with porting the critical data.nat.basic without delaying for this.


Open in Gitpod

@kim-em kim-em added the awaiting-review The author would like community review of the PR label Nov 29, 2022
@kim-em kim-em changed the title chore: split even_odd_rec out of data.nat.basic chore(data/nat/basic): split out even_odd_rec Nov 29, 2022
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Nov 29, 2022

✌️ semorrison can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Nov 29, 2022
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Nov 29, 2022

bors p=10

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Nov 29, 2022
bors bot pushed a commit that referenced this pull request Nov 29, 2022
`even_odd_rec` is annoying to port, because it is thoroughly tangled with `bit0` and `bit1`. It isn't used in mathlib, so I'm dumping it into its own file so we can get on with porting the critical `data.nat.basic` without delaying for this.




Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Nov 30, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(data/nat/basic): split out even_odd_rec [Merged by Bors] - chore(data/nat/basic): split out even_odd_rec Nov 30, 2022
@bors bors bot closed this Nov 30, 2022
@bors bors bot deleted the even_odd_rec branch November 30, 2022 02:19
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Nov 30, 2022
This is based on a *future* version of mathlib3, once leanprover-community/mathlib3#17763 and leanprover-community/mathlib3#17759 have landed.

- [x] depends on #727
- [x] depends on #730

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants