Skip to content

[Merged by Bors] - feat: port Order.SymmDiff#842

Closed
dupuisf wants to merge 16 commits intomasterfrom
dupuisf/order/symm_diff
Closed

[Merged by Bors] - feat: port Order.SymmDiff#842
dupuisf wants to merge 16 commits intomasterfrom
dupuisf/order/symm_diff

Conversation

@dupuisf
Copy link
Copy Markdown
Contributor

@dupuisf dupuisf commented Dec 3, 2022

mathlib3 SHA: 6eb334bd8f3433d5b08ba156b8ec3e6af47e1904

@dupuisf dupuisf added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. labels Dec 3, 2022
@dupuisf dupuisf added awaiting-review and removed WIP Work in progress labels Dec 4, 2022
Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

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

Looks good, except that you'll need to change nearly every theorem name

@Ruben-VandeVelde Ruben-VandeVelde added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Dec 4, 2022
dupuisf and others added 6 commits December 4, 2022 09:38
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@dupuisf dupuisf added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Dec 4, 2022
Copy link
Copy Markdown
Member

@kbuzzard kbuzzard left a comment

Choose a reason for hiding this comment

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

This looks good to me. Thanks! I've left some comments.

@dupuisf
Copy link
Copy Markdown
Contributor Author

dupuisf commented Dec 5, 2022

Thanks! I think I got them all now...

@ChrisHughes24
Copy link
Copy Markdown
Member

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 5, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 6, 2022

bors merge

bors bot pushed a commit that referenced this pull request Dec 6, 2022
mathlib3 SHA: 6eb334bd8f3433d5b08ba156b8ec3e6af47e1904

Co-authored-by: Frédéric Dupuis <31101893+dupuisf@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Order.SymmDiff [Merged by Bors] - feat: port Order.SymmDiff Dec 6, 2022
@bors bors bot closed this Dec 6, 2022
@bors bors bot deleted the dupuisf/order/symm_diff branch December 6, 2022 00:55
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 7, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.divisibility.units`: leanprover-community/mathlib4#848
* `algebra.group.type_tags`: leanprover-community/mathlib4#832
* `algebra.group_with_zero.divisibility`: leanprover-community/mathlib4#870
* `algebra.hom.equiv.basic`: leanprover-community/mathlib4#835
* `algebra.order.group.defs`: leanprover-community/mathlib4#869
* `algebra.order.monoid.basic`: leanprover-community/mathlib4#872
* `algebra.order.monoid.cancel.basic`: leanprover-community/mathlib4#883
* `algebra.order.monoid.with_zero.defs`: leanprover-community/mathlib4#851
* `algebra.order.monoid.with_zero.basic`: leanprover-community/mathlib4#851
* `algebra.ring.divisibility`: leanprover-community/mathlib4#864
* `data.list.defs`: leanprover-community/mathlib4#803
* `data.sigma.order`: leanprover-community/mathlib4#887
* `group_theory.group_action.defs`: leanprover-community/mathlib4#854
* `order.heyting.boundary`: leanprover-community/mathlib4#844
* `order.hom.basic`: leanprover-community/mathlib4#804
* `order.symm_diff`: leanprover-community/mathlib4#842



Co-authored-by: Johan Commelin <johan@commelin.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mathlib-port This is a port of a theory file from mathlib. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants