Skip to content

[Merged by Bors] - chore: remove some porting notes in sphere#12059

Closed
adomani wants to merge 2 commits intomasterfrom
adomani/restore_sphere_proof
Closed

[Merged by Bors] - chore: remove some porting notes in sphere#12059
adomani wants to merge 2 commits intomasterfrom
adomani/restore_sphere_proof

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Apr 11, 2024

Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.


Open in Gitpod

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@adomani adomani added awaiting-review porting-notes Mathlib3 to Mathlib4 porting notes. labels Apr 11, 2024
Copy link
Copy Markdown
Contributor

@grunweg grunweg left a comment

Choose a reason for hiding this comment

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

Looks like an improvement to me. Thanks for doing this!
(I'm low-to-medium confident on this one.)

Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

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 Apr 11, 2024
@mattrobball
Copy link
Copy Markdown
Contributor

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, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 11, 2024
Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 11, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: remove some porting notes in sphere [Merged by Bors] - chore: remove some porting notes in sphere Apr 11, 2024
@mathlib-bors mathlib-bors bot closed this Apr 11, 2024
@mathlib-bors mathlib-bors bot deleted the adomani/restore_sphere_proof branch April 11, 2024 15:29
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.
atarnoam pushed a commit that referenced this pull request Apr 16, 2024
Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.
callesonne pushed a commit that referenced this pull request Apr 22, 2024
Found after examining the bug fixes in #12054, although these improvements are unrelated to the test PR.
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. porting-notes Mathlib3 to Mathlib4 porting notes. 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