Skip to content

[Merged by Bors] - chore: Separate algebraic list lemmas - #12836

Closed
YaelDillies wants to merge 10 commits into
masterfrom
move_list_ring_big_ops
Closed

[Merged by Bors] - chore: Separate algebraic list lemmas#12836
YaelDillies wants to merge 10 commits into
masterfrom
move_list_ring_big_ops

Conversation

@YaelDillies

@YaelDillies YaelDillies commented May 12, 2024

Copy link
Copy Markdown
Contributor

Resplit Algebra.BigOperators.List.Basic and Algebra.BigOperators.List.Lemmas into two new files:

  • Algebra.BigOperators.Group.List for lemmas that require group-like structures (Monoid, Group, ...)
  • Algebra.BigOperators.Ring.List for lemmas that require ring-like structures (MonoidWithZero, Ring, ...)

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


Open in Gitpod

The lemmas can easily move to `Algebra.Group.Nat` and `Algebra.Group.Int`, `Algebra.Ring.Int` respectively.
Resplit `Algebra.BigOperators.List.Basic` and `Algebra.BigOperators.List.Lemmas` into two new files:
* `Algebra.BigOperators.Group.List` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.List` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #11855 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
@YaelDillies YaelDillies added blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) 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 12, 2024
Comment thread Mathlib/Algebra/BigOperators/Ring/List.lean Outdated
YaelDillies and others added 2 commits May 12, 2024 15:37
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@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 12, 2024
@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 13, 2024
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label May 15, 2024
@ghost

ghost commented May 15, 2024

Copy link
Copy Markdown

This PR/issue depends on:

@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 16, 2024
@github-actions

Copy link
Copy Markdown

No declarations were harmed in the making of this PR! 🐙


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>

@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 16, 2024
@riccardobrasca

Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions Bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels May 16, 2024
mathlib-bors Bot pushed a commit that referenced this pull request May 16, 2024
Resplit `Algebra.BigOperators.List.Basic` and `Algebra.BigOperators.List.Lemmas` into two new files:
* `Algebra.BigOperators.Group.List` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.List` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #11855 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
@mathlib-bors

mathlib-bors Bot commented May 16, 2024

Copy link
Copy Markdown
Contributor

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors Bot changed the title chore: Separate algebraic list lemmas [Merged by Bors] - chore: Separate algebraic list lemmas May 16, 2024
@mathlib-bors mathlib-bors Bot closed this May 16, 2024
@mathlib-bors
mathlib-bors Bot deleted the move_list_ring_big_ops branch May 16, 2024 16:17
callesonne pushed a commit that referenced this pull request May 16, 2024
Resplit `Algebra.BigOperators.List.Basic` and `Algebra.BigOperators.List.Lemmas` into two new files:
* `Algebra.BigOperators.Group.List` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.List` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #11855 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
grunweg pushed a commit that referenced this pull request May 17, 2024
Resplit `Algebra.BigOperators.List.Basic` and `Algebra.BigOperators.List.Lemmas` into two new files:
* `Algebra.BigOperators.Group.List` for lemmas that require group-like structures (`Monoid`, `Group`, ...)
* `Algebra.BigOperators.Ring.List` for lemmas that require ring-like structures (`MonoidWithZero`, `Ring`, ...)

Add `assert_not_exists Ring` in the former. Once #11855 lands, it will be strenghtenable to `assert_not_exists MonoidWithZero`.
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