Code review comment for lp:~leonardr/launchpad/toggle-representation-cache

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

In lib/lp/services/memcache/doc/restful-cache.txt you're missing an asterisk on the first line.

It'd be nice to have docstrings for the methods in MemcachedStormRepresentationCache, even if they just say "See `BaseRepositoryCache`."

review: Approve (code)

« Back to merge proposal