Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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!

@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move source under src/

3 participants