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] - fix(algebra.group.ulift): fix to_additive name for cancel_comm_monoid #18268

Closed
wants to merge 1 commit into from

Conversation

rwbarton
Copy link
Collaborator

@semorrison
Copy link
Collaborator

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Mar 29, 2023
@bors
Copy link

bors bot commented Mar 30, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix(algebra.group.ulift): fix to_additive name for cancel_comm_monoid [Merged by Bors] - fix(algebra.group.ulift): fix to_additive name for cancel_comm_monoid Mar 30, 2023
@bors bors bot closed this Mar 30, 2023
@bors bors bot deleted the rwbarton-patch-1 branch March 30, 2023 03:40
@vihdzp
Copy link
Collaborator

vihdzp commented Mar 30, 2023

Does anything need to happen on the Lean 4 side other than a SHA update?

bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 10, 2023
* `algebra.group.ulift`: this is a change that was already backported in leanprover-community/mathlib#18268 to match mathlib4, only the SHA needs to be updated.

* `set_theory.cardinal.basic`: the changes from leanprover-community/mathlib#18771 were already forward-ported in #3343, but the sha was not updated. Here I've also inlined all `by`s and generally tidied up the formatting.

* `order.rel_iso.basic` (leanprover-community/mathlib#18699, leanprover-community/mathlib#18758) is now fully forward-ported too.



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants