Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - refactor(algebra/group/defs): use is_left_cancel_mul etc#17884

Closed
urkud wants to merge 28 commits intomasterfrom
YK-is-mul-cancel
Closed

[Merged by Bors] - refactor(algebra/group/defs): use is_left_cancel_mul etc#17884
urkud wants to merge 28 commits intomasterfrom
YK-is-mul-cancel

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Dec 10, 2022

The Lean 4 version is here: leanprover-community/mathlib4#945.


I can port this to mathlib4 over the weekend.

Open in Gitpod

@urkud urkud added awaiting-review The author would like community review of the PR t-algebra Algebra (groups, rings, fields etc) labels Dec 10, 2022
@eric-wieser eric-wieser added the modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. label Dec 10, 2022
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

LGTM, as long as you're happy to deal with porting this

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Dec 10, 2022

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

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Dec 10, 2022
@urkud
Copy link
Copy Markdown
Member Author

urkud commented Dec 10, 2022

@eric-wieser Should I merge this now, or wait till the Lean 4 version is merged?

@eric-wieser
Copy link
Copy Markdown
Member

I think that's a question for the #mathlib4 stream

bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Dec 10, 2022
@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

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 Dec 10, 2022
bors bot pushed a commit that referenced this pull request Dec 10, 2022
@bors
Copy link
Copy Markdown

bors bot commented Dec 10, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(algebra/group/defs): use is_left_cancel_mul etc [Merged by Bors] - refactor(algebra/group/defs): use is_left_cancel_mul etc Dec 10, 2022
@bors bors bot closed this Dec 10, 2022
@bors bors bot deleted the YK-is-mul-cancel branch December 10, 2022 22:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

delegated The PR author may merge after reviewing final suggestions. modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) t-algebra Algebra (groups, rings, fields etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants