search

heremaps / xyz-spaces-python

Manage your XYZ Hub or HERE Data Hub spaces from Python.
https://xyz-spaces-python.readthedocs.io/
Apache License 2.0
30 stars 11 forks source link

Exclude tests submodules from package #121

Closed minff closed 3 years ago

minff commented 3 years ago

With current setup.py, pip install would still install "tests" package at the same level as xyzspaces package. Added wildcard pattern to exclude it

sackh commented 3 years ago

Thanks @minff, also you will have to remove below line https://github.com/heremaps/xyz-spaces-python/blob/b1b33b0ac55f017127d7153f4a01852a21e0648c/MANIFEST.in#L10

deeplook commented 3 years ago

I think that was on purpose for xyzspaces when I wrote it at the beginning, so people could also run the tests easily without the need to clone the whole repo. At least I think that was the reason. But if that's an issue with including it in the QGis-Plugin feel free to remove them...

minff commented 3 years ago

@deeplook , it is not really an issues with the QGIS plugin, but I find it strange when there is a tests package after pip install. If we want to include tests with the published package we could include it as submodule xyzspaces.tests imo.