Skip to content
New issue

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

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

Already on GitHub? # to your account

Move source to src/ #110

Merged
merged 5 commits into from
Dec 30, 2024
Merged

Move source to src/ #110

merged 5 commits into from
Dec 30, 2024

Conversation

sgillies
Copy link
Member

Resolves #109.

@sgillies sgillies added this to the 3.0 milestone Dec 27, 2024
@sgillies sgillies self-assigned this Dec 27, 2024
Copy link
Contributor

@mwtoews mwtoews left a comment

Choose a reason for hiding this comment

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

Changes look good!

tox.ini Outdated Show resolved Hide resolved
@mwtoews
Copy link
Contributor

mwtoews commented Dec 27, 2024

One last change:

diff --git a/README.rst b/README.rst
index 94737b8..ff58fce 100644
--- a/README.rst
+++ b/README.rst
@@ -16,3 +16,3 @@ Matrices describing 2D affine transformation of the plane.
 The Affine package is derived from Casey Duncan's Planar package. Please see
-the copyright statement in `affine/__init__.py <affine/__init__.py>`__.
+the copyright statement in `affine/src/__init__.py <affine/src/__init__.py>`__.
 

@sgillies sgillies mentioned this pull request Dec 28, 2024
@mwtoews
Copy link
Contributor

mwtoews commented Dec 29, 2024

Sorry to go back on my review, but another suggestion to consider is to move affine/__init__.pyaffine.py since this project is neatly contained in one module with no plans to expand any further (as far as I'm aware).

See #112 to this idea.

@sgillies sgillies merged commit b08075e into main Dec 30, 2024
7 checks passed
@sgillies sgillies deleted the issue109 branch December 30, 2024 02:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move source under src/
2 participants