Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - refactor(geometry/euclidean/oriented_angle): split file #17134

Closed
wants to merge 1 commit into from

Conversation

jsm28
Copy link
Collaborator

@jsm28 jsm28 commented Oct 24, 2022

Split geometry.euclidean.oriented_angle up into
geometry.euclidean.angle.oriented.basic,
geometry.euclidean.angle.oriented.affine and
geometry.euclidean.angle.sphere (the last only has a few results at present but is intended to get a lot more, both oriented results and unoriented results deduced from the oriented ones).


Whichever PR of this one and #17094 goes in first, the other will then need a minor update (since that PR modifies proofs that this PR moves to a different file).


Open in Gitpod

Split `geometry.euclidean.oriented_angle` up into
`geometry.euclidean.angle.oriented.basic`,
`geometry.euclidean.angle.oriented.affine` and
`geometry.euclidean.angle.sphere` (the last only has a few results at
present but is intended to get a lot more, both oriented results and
unoriented results deduced from the oriented ones).

---

Whichever PR of this one and #17094 goes in first, the other will then
need a minor update (since that PR modifies proofs that this PR
moves to a different file).
@jsm28 jsm28 added the awaiting-review The author would like community review of the PR label Oct 24, 2022
@urkud
Copy link
Member

urkud commented Oct 29, 2022

Are there any API/proof changes in this PR? If no, then
bors d+

@bors
Copy link

bors bot commented Oct 29, 2022

✌️ jsm28 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 Oct 29, 2022
@jsm28
Copy link
Collaborator Author

jsm28 commented Oct 29, 2022

There are no API or proof changes.

bors r+

bors bot pushed a commit that referenced this pull request Oct 29, 2022
Split `geometry.euclidean.oriented_angle` up into
`geometry.euclidean.angle.oriented.basic`,
`geometry.euclidean.angle.oriented.affine` and
`geometry.euclidean.angle.sphere` (the last only has a few results at present but is intended to get a lot more, both oriented results and unoriented results deduced from the oriented ones).
@bors
Copy link

bors bot commented Oct 29, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(geometry/euclidean/oriented_angle): split file [Merged by Bors] - refactor(geometry/euclidean/oriented_angle): split file Oct 29, 2022
@bors bors bot closed this Oct 29, 2022
@bors bors bot deleted the jsm28/oriented_angle_split branch October 29, 2022 05:03
@eric-wieser eric-wieser added the hacktoberfest-accepted Without this label hacktoberfest is scared off by bors label Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. hacktoberfest-accepted Without this label hacktoberfest is scared off by bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants