Skip to content

[Merged by Bors] - feat: port Algebra.Order.Hom.Basic#627

Closed
riccardobrasca wants to merge 9 commits intomasterfrom
RB/order_hom
Closed

[Merged by Bors] - feat: port Algebra.Order.Hom.Basic#627
riccardobrasca wants to merge 9 commits intomasterfrom
RB/order_hom

Conversation

@riccardobrasca
Copy link
Copy Markdown
Member

@riccardobrasca riccardobrasca commented Nov 17, 2022

mathlib3 sha: 17ef379e997badd73e5eabb4d38f11919ab3c4b3

@riccardobrasca riccardobrasca added help-wanted The author needs attention to resolve issues WIP Work in progress mathlib-port This is a port of a theory file from mathlib. and removed help-wanted The author needs attention to resolve issues labels Nov 17, 2022
@riccardobrasca riccardobrasca added awaiting-review and removed WIP Work in progress labels Nov 18, 2022
variable {ι F α β γ δ : Type _}

/-- `NonNegHomClass F α β` states that `F` is a type of nonnegative morphisms. -/
class NonNegHomClass (F : Type _) (α β : outParam (Type _)) [outParam (Zero β)] [outParam (LE β)]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lean4 nightly 2022-11-20 will remove the need for the outParams on typeclasses. Could you leave a porting note to that effect?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I will fix this when possible, thanks!

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 20, 2022

Besides the comment issues:

bors d+

Please make sure to update the port-status wiki page when you merge.

@bors
Copy link
Copy Markdown

bors bot commented Nov 20, 2022

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

@riccardobrasca
Copy link
Copy Markdown
Member Author

bors merge

bors bot pushed a commit that referenced this pull request Nov 20, 2022
mathlib3 sha: 17ef379e997badd73e5eabb4d38f11919ab3c4b3
@bors
Copy link
Copy Markdown

bors bot commented Nov 20, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Order.Hom.Basic [Merged by Bors] - feat: port Algebra.Order.Hom.Basic Nov 20, 2022
@bors bors bot closed this Nov 20, 2022
@bors bors bot deleted the RB/order_hom branch November 20, 2022 10:39
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants