-
Notifications
You must be signed in to change notification settings - Fork 485
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
Move to src layout #1146
Move to src layout #1146
Conversation
191835a
to
1c72b29
Compare
Move the project to the src layout to ensure we are testing on the generated package rather than the checked out source tree.
Move tests folder to the top of the source checkout and remove it from the package. We are still including the tests in the sdist in case downstream package managers want to test out their packaging.
@pganssle this should be ready now. |
@hroncok FYI, we've changed the layout which may change how tests and such are run. |
This is probably been taken into account but I thought I would still mention it. After |
I usually just go to the parent commit. Also note that github is smart enough to show you the relevant change in the file, see https://github.com/dateutil/dateutil/blame/master/src/dateutil/__init__.py. There is also a tag for the prev version, that can be used for that purposed as well. |
Is it possible to make new release with that commit? 🤔 |
Move the project to the src layout to ensure we are testing on the
generated package rather than the checked out source tree.
Summary of changes
Closes
Pull Request Checklist