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
Severity
High
Location
File:
README.mdLines: 210-213
Description
The example in the quick start section shows:
However, line 213 shows
LruTPolicyas 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:
This creates a poor first impression and immediate barrier to adoption.
Correct Example
Proposed Fix
Update line 213 and any other examples in README.md that show the old usage pattern.
Additional Context