Skip to content

[Merged by Bors] - feat: port Order.RelIso.Group#813

Closed
zeramorphic wants to merge 3 commits intomasterfrom
Order.RelIso.Group
Closed

[Merged by Bors] - feat: port Order.RelIso.Group#813
zeramorphic wants to merge 3 commits intomasterfrom
Order.RelIso.Group

Conversation

@zeramorphic
Copy link
Copy Markdown
Collaborator

mathlib3 SHA: 62a5626868683c104774de8d85b9855234ac807c

I would appreciate advice on naming lemmas about coercions, the porting guide doesn't say what to do.

Signed-off-by: thirdsgames <thirdsgames2018@gmail.com>
Signed-off-by: thirdsgames <thirdsgames2018@gmail.com>
@zeramorphic zeramorphic added awaiting-review mathlib-port This is a port of a theory file from mathlib. labels Dec 1, 2022
Signed-off-by: thirdsgames <thirdsgames2018@gmail.com>
@ChrisHughes24
Copy link
Copy Markdown
Member

ChrisHughes24 commented Dec 1, 2022

The convention I have already used was to use coe in the names of lemmas about coercions. Not much thought went into this however, but I would guess just leaving the names with coe is is the best for now?

@ChrisHughes24
Copy link
Copy Markdown
Member

The convention I have already used was to use coe in the names of lemmas about coercions. Not much thought went into this however, but I would guess just leaving the names with coe is is the best for now?

Some other PRs agreed with your convention however. I think we'll stick to your convention.

@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 1, 2022
bors bot pushed a commit that referenced this pull request Dec 1, 2022
mathlib3 SHA: 62a5626868683c104774de8d85b9855234ac807c

I would appreciate advice on naming lemmas about coercions, the porting guide doesn't say what to do.
@bors
Copy link
Copy Markdown

bors bot commented Dec 1, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Order.RelIso.Group [Merged by Bors] - feat: port Order.RelIso.Group Dec 1, 2022
@bors bors bot closed this Dec 1, 2022
@bors bors bot deleted the Order.RelIso.Group branch December 1, 2022 16:08
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 2, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.order.sub.canonical`: leanprover-community/mathlib4#814
* `category_theory.category.Rel`: leanprover-community/mathlib4#822
* `category_theory.category.basic`: leanprover-community/mathlib4#749
* `category_theory.functor.basic`: leanprover-community/mathlib4#749
* `category_theory.functor.category`: leanprover-community/mathlib4#749
* `category_theory.functor.functorial`: leanprover-community/mathlib4#822
* `category_theory.isomorphism`: leanprover-community/mathlib4#749
* `category_theory.natural_transformation`: leanprover-community/mathlib4#749
* `category_theory.thin`: leanprover-community/mathlib4#822
* `combinatorics.quiver.basic`: leanprover-community/mathlib4#749
* `combinatorics.quiver.path`: leanprover-community/mathlib4#811
* `data.psigma.order`: leanprover-community/mathlib4#815
* `data.stream.defs`: leanprover-community/mathlib4#665
* `order.boolean_algebra`: leanprover-community/mathlib4#794
* `order.heyting.basic`: leanprover-community/mathlib4#793
* `order.rel_iso.group`: leanprover-community/mathlib4#813
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 3, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.order.sub.canonical`: leanprover-community/mathlib4#814
* `category_theory.category.Rel`: leanprover-community/mathlib4#822
* `category_theory.category.basic`: leanprover-community/mathlib4#749
* `category_theory.functor.basic`: leanprover-community/mathlib4#749
* `category_theory.functor.category`: leanprover-community/mathlib4#749
* `category_theory.functor.functorial`: leanprover-community/mathlib4#822
* `category_theory.isomorphism`: leanprover-community/mathlib4#749
* `category_theory.natural_transformation`: leanprover-community/mathlib4#749
* `category_theory.thin`: leanprover-community/mathlib4#822
* `combinatorics.quiver.basic`: leanprover-community/mathlib4#749
* `combinatorics.quiver.path`: leanprover-community/mathlib4#811
* `data.psigma.order`: leanprover-community/mathlib4#815
* `data.stream.defs`: leanprover-community/mathlib4#665
* `order.boolean_algebra`: leanprover-community/mathlib4#794
* `order.heyting.basic`: leanprover-community/mathlib4#793
* `order.rel_iso.group`: leanprover-community/mathlib4#813
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.

2 participants