Skip to content

[Merged by Bors] - feat: port Algebra.Opposites#644

Closed
mcdoll wants to merge 11 commits intomasterfrom
mcdoll/algebra_opp
Closed

[Merged by Bors] - feat: port Algebra.Opposites#644
mcdoll wants to merge 11 commits intomasterfrom
mcdoll/algebra_opp

Conversation

@mcdoll
Copy link
Copy Markdown
Member

@mcdoll mcdoll commented Nov 18, 2022

mathlib SHA: fd47bdf09e90f553519c712378e651975fe8c829

@mcdoll mcdoll added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. labels Nov 18, 2022
@mcdoll mcdoll changed the title feat: port algebra.opposites feat: port Algebra.Opposites Nov 18, 2022
@Ruben-VandeVelde Ruben-VandeVelde added awaiting-review and removed WIP Work in progress labels Nov 21, 2022
Copy link
Copy Markdown
Contributor

@j-loreaux j-loreaux 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 to me aside from the minor issues mentioned below. Is it necessary to #align the additive versions? I would expect that it is (although I also suspect that mathport got most of these right)

@j-loreaux j-loreaux added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Nov 21, 2022
@mcdoll mcdoll added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Nov 24, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 24, 2022

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Nov 24, 2022

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

@kim-em kim-em added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Nov 24, 2022
@mcdoll
Copy link
Copy Markdown
Member Author

mcdoll commented Nov 24, 2022

bors merge

bors bot pushed a commit that referenced this pull request Nov 24, 2022
mathlib SHA: fd47bdf09e90f553519c712378e651975fe8c829

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Nov 24, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Opposites [Merged by Bors] - feat: port Algebra.Opposites Nov 24, 2022
@bors bors bot closed this Nov 24, 2022
@bors bors bot deleted the mcdoll/algebra_opp branch November 24, 2022 21:55
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Nov 25, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.char_zero.defs`: leanprover-community/mathlib4#661
* `algebra.group.inj_surj`: leanprover-community/mathlib4#707
* `algebra.group.order_synonym`: leanprover-community/mathlib4#651
* `algebra.group.units`: leanprover-community/mathlib4#549
* `algebra.hierarchy_design`: leanprover-community/mathlib4#657
* `algebra.opposites`: leanprover-community/mathlib4#644
* `algebra.quotient`: leanprover-community/mathlib4#643
* `algebra.ring.defs`: leanprover-community/mathlib4#655
* `algebra.ring.order_synonym`: leanprover-community/mathlib4#671
* `control.equiv_functor`: leanprover-community/mathlib4#649
* `data.dlist.basic`: leanprover-community/mathlib4#616
* `data.int.cast.basic`: leanprover-community/mathlib4#670
* `data.lazy_list`: leanprover-community/mathlib4#686
* `data.list.lex`: leanprover-community/mathlib4#672
* `data.option.n_ary`: leanprover-community/mathlib4#656
* `data.sigma.lex`: leanprover-community/mathlib4#646
* `data.ulift`: leanprover-community/mathlib4#703
* `group_theory.eckmann_hilton`: leanprover-community/mathlib4#626
* `logic.equiv.basic`: leanprover-community/mathlib4#631
* `order.iterate`: leanprover-community/mathlib4#648
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). mathlib-port This is a port of a theory file from mathlib.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants