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

[Merged by Bors] - refactor(data/set/finite): reduce imports#18245

Closed
urkud wants to merge 3 commits intomasterfrom
YK-set-finite-import
Closed

[Merged by Bors] - refactor(data/set/finite): reduce imports#18245
urkud wants to merge 3 commits intomasterfrom
YK-set-finite-import

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Jan 20, 2023


Open in Gitpod

@urkud urkud requested a review from a team as a code owner January 20, 2023 14:45
@urkud urkud added t-algebra Algebra (groups, rings, fields etc) t-measure-probability Measure theory / Probability theory modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. awaiting-review The author would like community review of the PR labels Jan 20, 2023
Copy link
Copy Markdown
Collaborator

@vihdzp vihdzp left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@urkud urkud requested a review from sgouezel January 20, 2023 17:01
Copy link
Copy Markdown
Collaborator

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

@urkud
Copy link
Copy Markdown
Member Author

urkud commented Jan 22, 2023

Note that data.set.finite is already ported, so I'm waiting for a delegate, not merge here. I'm going to forward-port the rest of this PR soon.

@YaelDillies
Copy link
Copy Markdown
Collaborator

I know, but I still don't have maintainer delegate 😦

But also note that it's acceptable to merge this PR now that you've opened a matching one, even if the matching one was not approved.

@sgouezel
Copy link
Copy Markdown
Collaborator

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Jan 22, 2023

✌️ 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.

@github-actions github-actions bot 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 Jan 22, 2023
@urkud
Copy link
Copy Markdown
Member Author

urkud commented Jan 22, 2023

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 Jan 22, 2023
bors bot pushed a commit that referenced this pull request Jan 22, 2023
* Add `eq_or_eq_or_eq_of_forall_not_lt_lt`, `finite.of_forall_not_lt_lt`, `set.finite_of_forall_not_lt_lt` (replacing `set.finite_of_forall_between_eq_endpoints`), and `set.finite_of_forall_not_lt_lt'`.
* Import `data.finset.basic` instead of `data.finset.sort` in `data.set.finite`.
* Forward-ported in leanprover-community/mathlib4#1738
@bors
Copy link
Copy Markdown

bors bot commented Jan 22, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(data/set/finite): reduce imports [Merged by Bors] - refactor(data/set/finite): reduce imports Jan 22, 2023
@bors bors bot closed this Jan 22, 2023
@bors bors bot deleted the YK-set-finite-import branch January 22, 2023 21:26
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jan 25, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jan 25, 2023
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) t-measure-probability Measure theory / Probability theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants