-
Notifications
You must be signed in to change notification settings - Fork 298
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] - refactor(set_theory/ordinal/*): ordinal.succ
→ order.succ
#14243
Conversation
succ
lemmas
I'm reworking this to completely replace |
succ
lemmasordinal.succ
→ order.succ
/-- The ordinal successor is the smallest ordinal larger than `o`. | ||
It is defined as `o + 1`. -/ | ||
def succ (o : ordinal) : ordinal := o + 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to update the module docstrnig to reference order.succ
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I won't make the call on whether we actually want to do this (someone who's worked with ordinals should answer that), but assuming we do then this PR looks great.
There's one place where you've forgotten to update a module docstring; can you go through the docstrings looking for succ`
and add order.
prefixes where appropriate?
I think this is a good idea, since it means we can reuse the results from |
I should mention - I had to add a lot of explicit |
bors merge. This looks great! |
We inline the definition of `ordinal.succ` in the `succ_order` instance. This allows us to comfortably use all of the theorems about `order.succ` to our advantage.
Pull request successfully merged into master. Build succeeded: |
ordinal.succ
→ order.succ
ordinal.succ
→ order.succ
We inline the definition of
ordinal.succ
in thesucc_order
instance. This allows us to comfortably use all of the theorems aboutorder.succ
to our advantage.(edit by @eric-wieser: tagged as easy since I've reviewed the diff, all we need is a "yes, generalization is worth more than dot notation" opinion by a user of
ordinal
)