Skip to content

[Merged by Bors] - chore: Relocate big operator lemmas#9383

Closed
YaelDillies wants to merge 10 commits intomasterfrom
sum_pow
Closed

[Merged by Bors] - chore: Relocate big operator lemmas#9383
YaelDillies wants to merge 10 commits intomasterfrom
sum_pow

Conversation

@YaelDillies
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies commented Jan 1, 2024

A bunch of lemmas in Algebra.BigOperators.Ring were not about rings. This PR moves them along with some lemmas from Data.Fintype.BigOperators to their correct place.

I create a new file with the content from #6605 to avoid importing Fin material in finset files as a result.

From LeanAPAP


Open in Gitpod

A bunch of lemmas in `Algebra.BigOperators.Ring` were not about rings. This PR moves them along some lemmas from `Data.Fintype.BigOperators` to their correct place.

I create a new file with the content from #6605 to avoid importing `Fin` material in finset files as a result.

From LeanAPAP
@YaelDillies YaelDillies added awaiting-review t-algebra Algebra (groups, rings, fields, etc) t-order Order theory awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. labels Jan 1, 2024
YaelDillies and others added 8 commits January 1, 2024 12:27
@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 Jan 6, 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 Jan 7, 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 Jan 9, 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 Jan 10, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jan 10, 2024
A bunch of lemmas in `Algebra.BigOperators.Ring` were not about rings. This PR moves them along with some lemmas from `Data.Fintype.BigOperators` to their correct place.

I create a new file with the content from #6605 to avoid importing `Fin` material in finset files as a result.

From LeanAPAP
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 10, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: Relocate big operator lemmas [Merged by Bors] - chore: Relocate big operator lemmas Jan 10, 2024
@mathlib-bors mathlib-bors bot closed this Jan 10, 2024
@mathlib-bors mathlib-bors bot deleted the sum_pow branch January 10, 2024 11:30
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) t-order Order theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants