-
Notifications
You must be signed in to change notification settings - Fork 5
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
make ZServer a dependency for py2 only #4
Conversation
I wonder if we should just drop the requirement for all Python versions.
|
The (really ugly) That has to be bogus: no reasonable test needs to have the server running. |
@hannosch Is this package intended to run with Zope 4 thus should it be ported to Python 3? |
@ale-rt Running the test on Python 3.6 breaks with |
Oh, I see @icemac. |
I don't think we should merge as-is. Instead, we should fix the stupid test which starts a server at import time, and remove the |
@tseaver Do you think this is mergeable after my fixes? |
Fixes #6