-
Notifications
You must be signed in to change notification settings - Fork 722
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
Modernize build (to Hatch) #774
Conversation
ba02477
to
523991d
Compare
Hey @ofek, if you have time to take a peek – I think I'm getting something hopelessly wrong with the src layout stuff if
I kinda understand that since the |
Thanks @ofek ❤️ |
Happy to help! Thanks for using your free time to update projects 😄 |
5cfb81e
to
659d041
Compare
@PierreF Rebased, ready for review. |
Co-authored-by: Ofek Lev <[email protected]> Co-authored-by: Pierre Fersing <[email protected]> Signed-off-by: Aarni Koskela <[email protected]>
Signed-off-by: Aarni Koskela <[email protected]>
Thank for this work. Also thank @cclauss and @semuadmin for the other PR on the same subject. |
This is a reimplementation of #712 and #719.
The packages generated are practically similar – this actually uncovered that
tests/testsupport
wasn't being correctly packaged into thesdist
before, so tests probably couldn't run with that package.