pdftotree icon indicating copy to clipboard operation
pdftotree copied to clipboard

PyPI v0.5.0 sdist is missing test data

Open jayvdb opened this issue 3 years ago • 3 comments

pdftotree-0.5.0/tests> ls
__init__.py  test_basic.py  test_table_detection.py

As a reult, running the tests from the PyPI sdist isnt currently possible.

The usual/old solution is to create a MANIFEST.in , which can be generated using check-manifest , but there are newer ways and tools if you do not want another metadata file in the repo.

jayvdb avatar Mar 01 '21 02:03 jayvdb

I see your point. One concern I would have is that the test data can be quite large. I'm not sure we actually want to bundle that together. Is there a good best practice to follow here?

lukehsiao avatar Mar 19 '21 04:03 lukehsiao

Best practise is to distribute wheels which only include the minimum needed to use the package, and a sdist which is a version snaphot of the 'source' which includes whatever is considered useful, e.g. docs, tests, test data, etc.

jayvdb avatar Mar 19 '21 07:03 jayvdb

Ah, I see. That makes sense then.

Would you happen to have the time to put together a PR for this? If not, it may be some time before I can get around to looking into this. I'll need to read up on the trade-offs of MANIFEST.in compared to what the more recent methods are.

lukehsiao avatar Mar 19 '21 16:03 lukehsiao