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/angle/oriented/basic): split out rotations #18212

Closed
wants to merge 1 commit into from

Conversation

jsm28
Copy link
Collaborator

@jsm28 jsm28 commented Jan 18, 2023

geometry.euclidean.angle.oriented.basic is 1489 lines long. Reduce it to 1032 lines by splitting out the definition of and results about rotation to a separate file. (No results in this file involve rotation in their proofs unless it's also involved in the statement.) There are no changes to API or proofs.


Open in Gitpod

`geometry.euclidean.angle.oriented.basic` is 1489 lines long.  Reduce
it to 1032 lines by splitting out the definition of and results about
`rotation` to a separate file.  (No results in this file involve
`rotation` in their proofs unless it's also involved in the
statement.)  There are no changes to API or proofs.
@jsm28 jsm28 added awaiting-review The author would like community review of the PR t-euclidean-geometry Affine and axiomatic geometry labels Jan 18, 2023
Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

Thanks!

@bors
Copy link

bors bot commented Jan 18, 2023

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

@eric-wieser eric-wieser added the awaiting-CI The author would like to see what CI has to say before doing more work. label Jan 18, 2023
@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 18, 2023
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Jan 18, 2023
@jsm28
Copy link
Collaborator Author

jsm28 commented Jan 18, 2023

bors r+

bors bot pushed a commit that referenced this pull request Jan 18, 2023
#18212)

`geometry.euclidean.angle.oriented.basic` is 1489 lines long.  Reduce it to 1032 lines by splitting out the definition of and results about `rotation` to a separate file.  (No results in this file involve `rotation` in their proofs unless it's also involved in the statement.)  There are no changes to API or proofs.
@bors
Copy link

bors bot commented Jan 18, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(geometry/euclidean/angle/oriented/basic): split out rotations [Merged by Bors] - refactor(geometry/euclidean/angle/oriented/basic): split out rotations Jan 18, 2023
@bors bors bot closed this Jan 18, 2023
@bors bors bot deleted the jsm28/split_angle_rotation branch January 18, 2023 20:44
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. t-euclidean-geometry Affine and axiomatic geometry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants