GitHub user GregBestland opened a pull request:

    https://github.com/apache/tinkerpop/pull/552

    Remove tests from packages

    Installing your tests package as part of the setup.py is probably not a 
good idea.
    
    Most every python project has a tests module for unit/integration tests, 
this is not generally distributed/installed as part of the package. When we 
install a tests module as part of the gremlinpython package we end up creating 
a scenario where namespace collision can occur with projects that wish to 
leverage the gremlinpython api. The gremlinpython tests module will end up 
superseding their own local tests directory in the sys.path, this creates 
problems for them when they go to try to run their own unit/integration tests.  
If you do need to install it, i'd recommend giving it a unique name like 
"gremlinpythontests", so as to prevent a namespace collision with the more 
commonly used "tests" module.
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/GregBestland/tinkerpop master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/552.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #552
    
----
commit 52fa8c8c1951f1210106dcdfbb03dddbfe719bf3
Author: GregBestland <[email protected]>
Date:   2017-01-27T17:41:25Z

    Remove tests from packages

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to