Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - feat(set_theory/ordinal/arithmetic): Add missing instances for ordinal #14128

Closed
wants to merge 35 commits into from

Conversation

vihdzp
Copy link
Collaborator

@vihdzp vihdzp commented May 13, 2022

We add the following instances:

  • monoid_with_zero ordinal
  • no_zero_divisors ordinal
  • is_left_distrib_class ordinal
  • contravariant_class ordinal ordinal (swap (+)) (<)
  • is_antisymm ordinal (∣)

Let me know if there's other missing instances here, I'm not very familiar with the algebraic hierarchy in mathlib.

Open in Gitpod

@vihdzp vihdzp added easy < 20s of review time. See the lifecycle page for guidelines. awaiting-review The author would like community review of the PR labels May 13, 2022
@vihdzp vihdzp changed the title feat(set_theory/ordinal/arithmetic): Add monoid_with_zero instance for ordinal feat(set_theory/ordinal/arithmetic): Add missing instances for ordinal May 13, 2022
@vihdzp vihdzp removed the easy < 20s of review time. See the lifecycle page for guidelines. label May 13, 2022
@mathlib-dependent-issues-bot mathlib-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label May 14, 2022
@mathlib-dependent-issues-bot mathlib-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label May 28, 2022
@mathlib-dependent-issues-bot
Copy link
Collaborator

@fpvandoorn
Copy link
Member

This looks good. Thanks!

bors d+

(This might conflict with some other PRs I put on the queue, so I'm just delegating it)

@bors
Copy link

bors bot commented May 28, 2022

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

@github-actions github-actions bot added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels May 28, 2022
@vihdzp
Copy link
Collaborator Author

vihdzp commented May 28, 2022

bors r+

bors bot pushed a commit that referenced this pull request May 28, 2022
…al` (#14128)

We add the following instances:
- `monoid_with_zero ordinal`
- `no_zero_divisors ordinal`
- `is_left_distrib_class ordinal`
- `contravariant_class ordinal ordinal (swap (+)) (<)`
- `is_antisymm ordinal (∣)`
@bors
Copy link

bors bot commented May 28, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(set_theory/ordinal/arithmetic): Add missing instances for ordinal [Merged by Bors] - feat(set_theory/ordinal/arithmetic): Add missing instances for ordinal May 28, 2022
@bors bors bot closed this May 28, 2022
@bors bors bot deleted the ordinal_missing_instances branch May 28, 2022 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants