Skip to content

[Merged by Bors] - chore: forward port mathlib#17611, 18742, 18198, 18520#3389

Closed
Parcly-Taxel wants to merge 4 commits intomasterfrom
17611
Closed

[Merged by Bors] - chore: forward port mathlib#17611, 18742, 18198, 18520#3389
Parcly-Taxel wants to merge 4 commits intomasterfrom
17611

Conversation

@Parcly-Taxel
Copy link
Copy Markdown
Collaborator

@Parcly-Taxel Parcly-Taxel commented Apr 11, 2023

@Parcly-Taxel Parcly-Taxel added awaiting-review mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged labels Apr 11, 2023
@eric-wieser
Copy link
Copy Markdown
Member

eric-wieser commented Apr 11, 2023

Can you explain in the description why each of the SHA-only changes needs no forward-port?

@Parcly-Taxel
Copy link
Copy Markdown
Collaborator Author

Can you explain in the description why each of the SHA-only changes needs no forward-point?

Ah, done.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Apr 11, 2023

@Parcly-Taxel, please don't use numeric branch names: in some circumstances git is uncertain whether to interpret these as SHAs.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Apr 11, 2023

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Apr 11, 2023
bors bot pushed a commit that referenced this pull request Apr 11, 2023
SHA-only updates:
* leanprover-community/mathlib3#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib3#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib3#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

Substantative forward port:
* leanprover-community/mathlib3#18520



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Apr 11, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore: forward port mathlib#17611, 18742, 18198, 18520 [Merged by Bors] - chore: forward port mathlib#17611, 18742, 18198, 18520 Apr 11, 2023
@bors bors bot closed this Apr 11, 2023
@bors bors bot deleted the 17611 branch April 11, 2023 21:08
eric-wieser pushed a commit that referenced this pull request Apr 12, 2023
SHA-only updates:
* leanprover-community/mathlib3#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib3#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib3#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

Substantative forward port:
* leanprover-community/mathlib3#18520



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
@ericrbg
Copy link
Copy Markdown
Contributor

ericrbg commented Apr 18, 2023

https://leanprover-community.github.io/mathlib-port-status/out-of-sync

On the dashboard, the forward port for mathlib#18520 does not seem to have come through.

@eric-wieser
Copy link
Copy Markdown
Member

What do you mean by that? Also note that you have to write leanprover-community/mathlib3#18520

@ericrbg
Copy link
Copy Markdown
Contributor

ericrbg commented Apr 18, 2023

I realise now I got confused - I thought this fully forward-ported the connected components section, but this only does little bits of it. I'll work on the rest of that.

Edit: I see now also that that's not possible yet as noted in the PR description. Sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants