Skip to content

[Merged by Bors] - feat: Refactor ConnectedComponent#971

Closed
bottine wants to merge 8 commits intomasterfrom
bottine/Quiver/edge_reversal_refactor
Closed

[Merged by Bors] - feat: Refactor ConnectedComponent#971
bottine wants to merge 8 commits intomasterfrom
bottine/Quiver/edge_reversal_refactor

Conversation

@bottine
Copy link
Copy Markdown
Contributor

@bottine bottine commented Dec 13, 2022

… in order to sync with PR 17665 on mathlib3.

See leanprover-community/mathlib3#17665

… in order to sync with PR 17665 on mathlib3.
bottine and others added 3 commits December 13, 2022 09:41
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
@kim-em kim-em added the awaiting-author A reviewer has asked the author a question or requested changes. label Dec 13, 2022
…com/leanprover-community/mathlib4 into bottine/Quiver/edge_reversal_refactor
@bottine bottine added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Dec 13, 2022
@YaelDillies YaelDillies changed the title feat(Combinatorics/Quiver/*): Refactor ConnectedComponent feat: Refactor ConnectedComponent Dec 13, 2022
@bottine bottine requested a review from YaelDillies December 13, 2022 13:47
Copy link
Copy Markdown
Contributor

@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.

Let's wait for the mathlib PR to get in.

@bottine
Copy link
Copy Markdown
Contributor Author

bottine commented Dec 13, 2022

I'm currently changing the mathlib3 PR according to Yael's suggestion. Will update this one to synchronize when the mathlib3 one gets approved.

@bottine bottine added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Dec 13, 2022
@kim-em kim-em added awaiting-author A reviewer has asked the author a question or requested changes. and removed blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) awaiting-review labels Dec 13, 2022
@bottine bottine added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Dec 17, 2022
@bottine
Copy link
Copy Markdown
Contributor Author

bottine commented Dec 17, 2022

NB: I believe the troubles with universes that forced the mathlib3 definition of Quiver.Symmetrify to be a Sum rather than a Psum disappeared with mathlib4. I've been told that it's not ideal to do such changes during the port.
When is a good time to do it?

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 17, 2022

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Dec 17, 2022

✌️ bottine 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 This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Dec 17, 2022
@bottine
Copy link
Copy Markdown
Contributor Author

bottine commented Dec 17, 2022

bors r+

bors bot pushed a commit that referenced this pull request Dec 17, 2022
… in order to sync with PR 17665 on mathlib3.

See leanprover-community/mathlib3#17665

Co-authored-by: Rémi Bottinelli <bottine@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 17, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: Refactor ConnectedComponent [Merged by Bors] - feat: Refactor ConnectedComponent Dec 17, 2022
@bors bors bot closed this Dec 17, 2022
@bors bors bot deleted the bottine/Quiver/edge_reversal_refactor branch December 17, 2022 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants