Skip to content

[Merged by Bors] - chore(Algebra/Group): Do not import GroupWithZero#11202

Closed
YaelDillies wants to merge 4 commits intomasterfrom
group_no_zero
Closed

[Merged by Bors] - chore(Algebra/Group): Do not import GroupWithZero#11202
YaelDillies wants to merge 4 commits intomasterfrom
group_no_zero

Conversation

@YaelDillies
Copy link
Copy Markdown
Contributor

I am claiming that anything within the Algebra.Group folder should be additivisable, to the exception of MonoidHom.End maybe. This is not the case of NeZero, MonoidWithZero and MonoidWithZeroHom which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.


Open in Gitpod

I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
@YaelDillies YaelDillies added awaiting-review t-algebra Algebra (groups, rings, fields, etc) labels Mar 6, 2024
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Mar 7, 2024
mathlib-bors bot pushed a commit that referenced this pull request Mar 7, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 7, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Algebra/Group): Do not import GroupWithZero [Merged by Bors] - chore(Algebra/Group): Do not import GroupWithZero Mar 7, 2024
@mathlib-bors mathlib-bors bot closed this Mar 7, 2024
@mathlib-bors mathlib-bors bot deleted the group_no_zero branch March 7, 2024 06:34
kbuzzard pushed a commit that referenced this pull request Mar 12, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
utensil pushed a commit that referenced this pull request Mar 26, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
callesonne pushed a commit that referenced this pull request Apr 22, 2024
I am claiming that anything within the `Algebra.Group` folder should be additivisable, to the exception of `MonoidHom.End` maybe. This is not the case of `NeZero`, `MonoidWithZero` and `MonoidWithZeroHom` which were all imported to prove a few lemmas. Those lemmas are easily moved to another file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants