-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add recommendation for cached-property #46
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'd support adding this here... already there a few places i can think to use it
go-to-libraries.md
Outdated
|
||
### [`cached-property`](https://pypi.org/project/cached-property/) | ||
|
||
@piper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure why this is here
edit: i see, its tagged by suggester
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I figure it's good to know who was championing something. In theory git blame works for this but that's bad UX.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@piper suggests:
or
@piper recommends:
to avoid future questions of the same type :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe worth noting that it collides with __slots__
, so you have to choose one or the other.
go-to-libraries.md
Outdated
|
||
### [`cached-property`](https://pypi.org/project/cached-property/) | ||
|
||
@piper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@piper suggests:
or
@piper recommends:
to avoid future questions of the same type :)
FYI about Looks like there's a work-around... |
Uh oh, I foresee us forking and maintaining But yay that there's a solution! pydanny/cached-property#137 |
Add
cached-property
recommendation