Remove transitive dependency zope.interface from requirements.txt

[karyotakisg]

As part of our ongoing research on Python dependency management we noticed a potential improvement in your project’s dependency management.

Specifically, the transitive dependency zope.interface of package Twisted is specified as a requirement in the requirements.txt file, when in reality it is not needed.

This PR removes it from requirements.txt to let pip manage it automatically, which helps keeping the dependency list clean.

Hope this is helpful!

Best regards

[sibson]

Is there a PEP that states this is the best practice?

[karyotakisg]

Hi again,
Pip documentation states that it is useful to audit your project and remove any unnecessary or out of date requirements. By removing these you can reduce the complexity of the dependency tree, thus reducing opportunities for conflicts to occur [see: pip-audit your requirements]

Moreover, in a case where Twisted stops using zope.interface as a dependency, then zope.interface will be installed completely unnecessarily.

[pmhahn]

VNCdotool does not use zope.interface directly itself; it only gets used indirectly inside Twisted. Therefore declaring an explicit dependency on it looks pointless.
I'm in favor of removing this dependency.