Repository navigation
Use jaraco.classes for properties - #588
Conversation
|
Maybe reexport them for the time being for backward compatibility? (I'm tempted to replace "Maybe" with "At least", but I feel that makes me sounds too entitled.) |
I can do that - make it an optional dependency for the time being. |
e110fb2 to
3c38fa4
Compare
|
Isn't that kinda much to add another dependency just for these two methods? |
|
Yes and no.
Principles I roughly follow for dependencies:
I think implicit in your question is that adding a dependency is a cost that should be taken seriously. Can you articulate or link to a resource that explains those costs? |
try:
from jaraco.compat import properties # pragma: no-cover
except ImportError:
from . import _properties_compat as properties # pragma: no-coverI think |
Today I was reviewing the implementation of
NonDataPropertyandClassPropertyonly to find that there were stale versions of the implementations inkeyring.util.propertiesthat represent old versions of the original implementations injaraco.classes.I'm tempted to simply depend on that code. Because it introduces a new dependency, I'm somewhat reluctant to accept the change. I'm putting it here for feedback and consideration.