Skip to content

[Merged by Bors] - chore: Split Algebra.Group.Prod - #12974

Closed
YaelDillies wants to merge 9 commits into
masterfrom
split_group_prod
Closed

[Merged by Bors] - chore: Split Algebra.Group.Prod#12974
YaelDillies wants to merge 9 commits into
masterfrom
split_group_prod

Conversation

@YaelDillies

Copy link
Copy Markdown
Contributor

Move the non-additivisable declarations to a new file Algebra.GroupWithZero.Prod. Credit Eric for leanprover-community/mathlib3#6866 and myself for leanprover-community/mathlib3#10907.


Open in Gitpod

Move the non-additivisable declarations to a new file `Algebra.GroupWithZero.Prod`. Credit Eric for leanprover-community/mathlib3#6866 and myself for leanprover-community/mathlib3#10907.
@YaelDillies YaelDillies added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-algebra Algebra (groups, rings, fields, etc) labels May 17, 2024
@github-actions github-actions Bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label May 17, 2024
Comment thread Mathlib/Algebra/Group/Prod.lean Outdated
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 18, 2024
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 19, 2024
@github-actions

Copy link
Copy Markdown

+ instCommMonoidWithZero
+ instMonoidWithZero
+ instMulZeroClass
+ instMulZeroOneClass
+ instSemigroupWithZero
- instance [CommMonoidWithZero M] [CommMonoidWithZero N] : CommMonoidWithZero (M × N)
- instance [MonoidWithZero M] [MonoidWithZero N] : MonoidWithZero (M × N)
- instance [MulZeroClass M] [MulZeroClass N] : MulZeroClass (M × N)
- instance [MulZeroOneClass M] [MulZeroOneClass N] : MulZeroOneClass (M × N)
- instance [SemigroupWithZero M] [SemigroupWithZero N] : SemigroupWithZero (M × N)


You can run this locally as follows

## summary with just the declaration names:
./scripts/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

mathlib-bors Bot pushed a commit that referenced this pull request May 20, 2024
Resplit `Algebra.BigOperators.Multiset.Basic` and `Algebra.BigOperators.Multiset.Lemmas` into two new files:
* `Algebra.BigOperators.Group.Multiset` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.Multiset` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #12974 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maintainer merge

@github-actions

Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

@github-actions github-actions Bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label May 21, 2024

@jcommelin jcommelin left a comment

Copy link
Copy Markdown
Member

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 May 21, 2024
mathlib-bors Bot pushed a commit that referenced this pull request May 21, 2024
Move the non-additivisable declarations to a new file `Algebra.GroupWithZero.Prod`. Credit Eric for leanprover-community/mathlib3#6866 and myself for leanprover-community/mathlib3#10907.
@mathlib-bors

mathlib-bors Bot commented May 21, 2024

Copy link
Copy Markdown
Contributor

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors Bot changed the title chore: Split Algebra.Group.Prod [Merged by Bors] - chore: Split Algebra.Group.Prod May 21, 2024
@mathlib-bors mathlib-bors Bot closed this May 21, 2024
@mathlib-bors
mathlib-bors Bot deleted the split_group_prod branch May 21, 2024 09:57
callesonne pushed a commit that referenced this pull request Jun 4, 2024
Resplit `Algebra.BigOperators.Multiset.Basic` and `Algebra.BigOperators.Multiset.Lemmas` into two new files:
* `Algebra.BigOperators.Group.Multiset` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.Multiset` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #12974 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
callesonne pushed a commit that referenced this pull request Jun 4, 2024
Move the non-additivisable declarations to a new file `Algebra.GroupWithZero.Prod`. Credit Eric for leanprover-community/mathlib3#6866 and myself for leanprover-community/mathlib3#10907.
js2357 pushed a commit that referenced this pull request Jun 18, 2024
Resplit `Algebra.BigOperators.Multiset.Basic` and `Algebra.BigOperators.Multiset.Lemmas` into two new files:
* `Algebra.BigOperators.Group.Multiset` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.Multiset` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #12974 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
js2357 pushed a commit that referenced this pull request Jun 18, 2024
Move the non-additivisable declarations to a new file `Algebra.GroupWithZero.Prod`. Credit Eric for leanprover-community/mathlib3#6866 and myself for leanprover-community/mathlib3#10907.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. 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.

3 participants