Skip to content

[Merged by Bors] - feat: port Algebra.Order.Group.Units#898

Closed
hrmacbeth wants to merge 3 commits intomasterfrom
hrmacbeth-algebra-order-group-units
Closed

[Merged by Bors] - feat: port Algebra.Order.Group.Units#898
hrmacbeth wants to merge 3 commits intomasterfrom
hrmacbeth-algebra-order-group-units

Conversation

@hrmacbeth
Copy link
Copy Markdown
Member

mathlib3 a95b16cbade0f938fc24abd05412bde1e84bab9b

Porting notes:

@hrmacbeth hrmacbeth added awaiting-review mathlib-port This is a port of a theory file from mathlib. labels Dec 7, 2022
@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 7, 2022
@bors
Copy link
Copy Markdown

bors bot commented Dec 7, 2022

Merge conflict.

@kim-em kim-em added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 7, 2022
@hrmacbeth hrmacbeth removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 7, 2022
@hrmacbeth
Copy link
Copy Markdown
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Dec 7, 2022
mathlib3 a95b16cbade0f938fc24abd05412bde1e84bab9b

Porting notes:
- the module docstring title in mathlib3 was incorrect (probably a relic of a hasty split), I have changed it here without backporting
- one elaboration issue, see [Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/elaboration.20failure.20in.20algebra.2Eorder.2Egroup.2Eunits)
- same issue with instance names as in #890, and I made the same choice about how to fix it
@bors
Copy link
Copy Markdown

bors bot commented Dec 7, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Order.Group.Units [Merged by Bors] - feat: port Algebra.Order.Group.Units Dec 7, 2022
@bors bors bot closed this Dec 7, 2022
@bors bors bot deleted the hrmacbeth-algebra-order-group-units branch December 7, 2022 16:47
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 8, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.euclidean_domain.defs`: leanprover-community/mathlib4#871
* `algebra.group.ext`: leanprover-community/mathlib4#850
* `algebra.group_power.basic`: leanprover-community/mathlib4#874
* `algebra.hom.equiv.units.basic`: leanprover-community/mathlib4#895
* `algebra.hom.equiv.units.group_with_zero`: leanprover-community/mathlib4#901
* `algebra.order.group.instances`: leanprover-community/mathlib4#890
* `algebra.order.group.order_iso`: leanprover-community/mathlib4#895
* `algebra.order.group.units`: leanprover-community/mathlib4#898
* `algebra.order.monoid.nat_cast`: leanprover-community/mathlib4#893
* `algebra.order.monoid.type_tags`: leanprover-community/mathlib4#873
* `algebra.order.monoid.units`: leanprover-community/mathlib4#873
* `algebra.order.zero_le_one`: leanprover-community/mathlib4#893
* `combinatorics.quiver.push`: leanprover-community/mathlib4#868
* `control.traversable.basic`: leanprover-community/mathlib4#788
* `data.sum.order`: leanprover-community/mathlib4#880
* `group_theory.group_action.option`: leanprover-community/mathlib4#884
* `group_theory.group_action.sigma`: leanprover-community/mathlib4#885
* `group_theory.group_action.sum`: leanprover-community/mathlib4#882
* `group_theory.group_action.units`: leanprover-community/mathlib4#878
* `order.antisymmetrization`: leanprover-community/mathlib4#876
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.

3 participants