search

clokep / django-render-block

Render the content of a specific block tag from a Django template.
ISC License
241 stars 23 forks source link

Include test templates in sdist tarball #30

Open Natureshadow opened 3 years ago

Natureshadow commented 3 years ago

Please include these templates and release a new (post) release to PyPI.

This is important to be able to reproducibly run the test suite in the Debian package.

clokep commented 3 years ago

While testing this I ended up with the tests directory installed into site-packages, which is generally unwanted.

This seems to match what Django has in their MANIFEST.in, but I don't seem to end up with their tests in site-packages -- maybe there's a missing config somewhere to control that?

ghost commented 2 years ago

I tried to rebase this branch on top of main and ran pyproject-build .. It correctly includes the tests folder in the sdist and not in the wheel.

clokep commented 5 months ago

While testing this I ended up with the tests directory installed into site-packages, which is generally unwanted.

Do you have any response to this? Is there updated guidance saying this is a good idea?