-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentation
Description
Severity
High
Location
File: README.md
Lines: 210-213
Description
The example in the quick start section shows:
lru_cache = Cache("my-first-lru-cache", LruTPolicy, factory)However, line 213 shows LruTPolicy as a class, not an instance. This contradicts the v0.7 breaking change documentation (lines 232-270) which states policies must be instantiated: LruTPolicy().
Impact
Users following the example will encounter:
TypeError: AbstractPolicy subclasses must be instantiated before passing to RedisFuncCache
This creates a poor first impression and immediate barrier to adoption.
Correct Example
# Policy must be instantiated (note the parentheses)
lru_cache = Cache("my-first-lru-cache", LruTPolicy(), factory)Proposed Fix
Update line 213 and any other examples in README.md that show the old usage pattern.
Additional Context
- Breaking change introduced in v0.7.0
- CHANGELOG.md correctly documents the change
- Only the quick start example needs updating
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentation