-
-
-
rbtools/api/cache.py (Diff revision 1) Col: 34 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
Add API caching support to rbtools.
Review Request #6504 — Created Oct. 24, 2014 and submitted
Add the
rbtools.api.cache
module. This module includes the
APICache
class, which allows the results ofHTTP GET
requests to
be cached locally to achieve faster lookup. TheAPICache
understands
the following headers:
Cache-Control
Pragma
Vary
Last-Modified
ETag
Expires
The APICache
is backed by a SQLite database in the user's cache
directory. If the cache cannot be accessed, the APICache
will pass
all requests to the specified urlopen
(defaulting to
urllib2.urlopen
) method.
The APICache
allows for a custom urlopen
function so that it can
be tested without making actual HTTP requests.
The APICache
stores the version of its schema in the database. If
the APICache
is instantiated and the versions of the schema in the
class doesn't match the version of the schema in the database, then
the current cache will be invalidted and the schema will be updated.
Add the ActiveState appdirs
module (at version 1.4.0) as
rbtools.utils.appdirs
. This module is used to find the user cache
directory in platform-dependant ways.
Add the rbtools.api.cache.Response
class which emulates the
behaviour of a urllib2
response. This class allows the read
method to be called more than once.
Add the rbtools.api.cache.CachedResponse
class, which is the result
of a cach entry being found the the APICache
. It also emulates the
behaviour of a urllib2
response.
Add the clear-cache
command. This command manually clears the API
cache.
Added unit tests for the APICache
.
Unit tests pass.
The following commands were tested and were successful:
rbt status
rbt post
rbt diff
rbt patch
rbt close
rbt clear-cache
Upon clearing the cache with clear-cache
, a new cache is created.
The first run of a command always creates cache entries (where
appropriate, i.e., not for PUT or POST requests).
The /api/
endpoint is successfully cached and the cached version is
used.
When using the patch
command for the second time on the same review
request, if the diff has not been updated, then the cached version of
the /api/review-requests/<id>/diff/<diff_revision>/
endpoint is
used.
This review request was posted with the APICache
enabled.
Description | From | Last Updated |
---|---|---|
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 34 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 5 E303 too many blank lines (2) |
reviewbot | |
Col: 13 E303 too many blank lines (3) |
reviewbot | |
Col: 9 E303 too many blank lines (2) |
reviewbot | |
Col: 5 E303 too many blank lines (2) |
reviewbot | |
Col: 80 E501 line too long (88 > 79 characters) |
reviewbot | |
Col: 32 E261 at least two spaces before inline comment |
reviewbot | |
Col: 5 E303 too many blank lines (2) |
reviewbot | |
Col: 1 W391 blank line at end of file |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
local variable 'kernel' is assigned to but never used |
reviewbot | |
undefined name 'kernal' |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
Col: 1 W293 blank line contains whitespace |
reviewbot | |
Col: 1 W293 blank line contains whitespace |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 9 E303 too many blank lines (2) |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Col: 23 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Switch these two. (cle before clo) |
david | |
I know that this module is part of the standard library, but is it always guaranteed to work? I don't … |
david | |
Since this will be going into master, and we're bumping the minimum python version, you can change this to just … |
david | |
except OSError as e: |
david | |
This can go away. |
david | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Can you alphabetize this list? |
david | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'json' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
These should each be using the :file: markup. (I know we don't have that above, but we should there as … |
chipx86 | |
We should maybe say the HTTP cache for the API. |
chipx86 | |
This and the doc page should match the command: clear-cache |
chipx86 | |
We can just import os. |
chipx86 | |
This isn't part of Python, so it must go in import group #2. |
chipx86 | |
This is a Python standard library module, so it can go in the group above. |
chipx86 | |
What's the reason for setting __slots__ here? Generally speaking, there's rarely a reason to actually set this. It can be … |
chipx86 | |
The function name and docstring sort of contradict each other. |
chipx86 | |
Might be worth mentioning that this is intended to be API-compatible with a urllib2's response object. Same with CachedResponse below. … |
chipx86 | |
Can this be a @staticmethod on ApiCache? |
chipx86 | |
Trailing period at the end of the log message. Variable on the next line. |
chipx86 | |
No blank line here. |
chipx86 | |
These could result in exceptions as well, right? Anything we should be catching here? Also, you could skip doing this … |
chipx86 | |
No blank line here. |
chipx86 | |
Unless we're explicitly needing to worry about the difference between None, empty strings, 0, etc., it's usually best to just … |
chipx86 | |
No blank line. |
chipx86 | |
In this particular case, it's fine to use \ instead of parens for line continuation. |
chipx86 | |
No blank line. |
chipx86 | |
"cacheable" |
chipx86 | |
No blank line. |
chipx86 | |
We return response in all of these cases. We should probably just do a single return response at the end. … |
chipx86 | |
Must be on one line. Doc generators will chop it off otherwise. |
chipx86 | |
I read this incorrectly at first. Can you add a small comment saying that we restore the old value below? … |
chipx86 | |
Blank line between these. |
chipx86 | |
expires can be a separate parameter. |
chipx86 | |
How about "The C locale is not available on this system. The 'Expires' header cannot be parsed." |
chipx86 | |
We should actually grab the old locale and set it, instead of calling resetlocale, in case the caller has actually … |
chipx86 | |
No blank line before elif and else. Same below. |
chipx86 | |
"posterity's" |
chipx86 | |
Should be on the same line. I'd rather not do the map/strip trick inline like this, and like the one … |
chipx86 | |
We don't need to check against None unless there's a reason to worry about empty strings and zero. I believe … |
chipx86 | |
We should probably catch errors for all these commands. |
chipx86 | |
Missing a trailing period. |
chipx86 | |
No blank line here. |
chipx86 | |
We use dumps to serialize, but are loading above with a custom split. dumps may be too heavy. Can we … |
chipx86 | |
"ensures" |
chipx86 | |
Swap these. |
chipx86 | |
Missing a docstring. |
chipx86 | |
How about "Deletes the HTTP cache used for the API." |
chipx86 | |
Rather than this, how about a method on ApiCache for deleting the cache? |
chipx86 | |
Parameters should go on the line after the log message. |
chipx86 | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 13 E303 too many blank lines (2) |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Should go up before logging. |
chipx86 | |
This blank line should be removed. |
chipx86 | |
""" on the next line. |
chipx86 | |
Instead of terminating the process, can we just not cache? RBTools is often run in places where it's locked down … |
chipx86 | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'urlopen' imported but unused |
reviewbot | |
undefined name 'urllib2' |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
'urlopen' imported but unused |
reviewbot | |
undefined name 'urllib2' |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Should catch sqlite3.Error here and die gracefully. |
brennie | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
For consistency with the RB database, mind if we use cache_info? |
chipx86 | |
And api_cache. |
chipx86 | |
"URL" |
chipx86 | |
"URL" |
chipx86 | |
"If-None-Match" |
chipx86 | |
No blank line. |
chipx86 | |
No blank line. |
chipx86 | |
These are only available in Python 2.7+. |
chipx86 | |
"API" |
chipx86 | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot | |
Col: 80 E501 line too long (96 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (93 > 79 characters) |
reviewbot | |
Col: 38 E261 at least two spaces before inline comment |
reviewbot | |
Col: 36 E261 at least two spaces before inline comment |
reviewbot | |
Col: 10 E261 at least two spaces before inline comment |
reviewbot | |
Col: 1 E303 too many blank lines (3) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (107 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (122 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (87 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (94 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (92 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (89 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (116 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (115 > 79 characters) |
reviewbot | |
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot | |
Col: 30 E128 continuation line under-indented for visual indent |
reviewbot | |
Col: 1 E265 block comment should start with '# ' |
reviewbot | |
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot | |
'win32com' imported but unused |
reviewbot | |
'windll' imported but unused |
reviewbot | |
'com' imported but unused |
reviewbot |
Change Summary:
Formatting.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+953 -9) |
-
Tool: Pyflakes Processed Files: rbtools/api/transport/sync.py rbtools/api/request.py rbtools/utils/appdirs.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/transport/sync.py rbtools/api/request.py rbtools/utils/appdirs.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 2) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Refactor ApiCache to use dependancy injection so that a mockup urlopen can be used. Fully flesh out behaviour. Write unit tests.
Summary: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Description: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Testing Done: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Commit: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Groups: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 3 (+1293 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/api/request.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/api/request.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 35.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
Change Summary:
Formatting
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 4 (+1294 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/api/request.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/api/request.py rbtools/api/cache.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 4) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Add
rbt clear-cache
command and its documentation.Code cleanup and documentation.
Description: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Commit: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 5 (+1371 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 34.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 5) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
Change Summary:
Formatting.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 6 (+1372 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 6) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
Change Summary:
Fix formatting and a typo.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 7 (+1371 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 7) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
Change Summary:
Add todo to description.
Summary: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Description: |
|
Change Summary:
Allow
ApiCache
to be invalidated.Move
ApiCache._insert_entry
andApiCache._update_entry
toApiCache._save_entry
which performs the funcationality of both.
Description: |
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Commit: |
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 8 (+1393 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 33.
-
rbtools/api/cache.py (Diff revision 8) Col: 23 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 8) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Formatting.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 9 (+1398 -3) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 9) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
-
-
-
rbtools/api/cache.py (Diff revision 9) I know that this module is part of the standard library, but is it always guaranteed to work? I don't want users to have to install anything other than python.
-
rbtools/api/cache.py (Diff revision 9) Since this will be going into master, and we're bumping the minimum python version, you can change this to just be
import json
and usejson.loads
andjson.dumps
-
-
-
Change Summary:
Fix issues in David's review.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 10 (+1392 -4) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 10) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
-
This change looks pretty good to me, but I'd like some details on manual testing of the individual rbtools commands.
Change Summary:
Write the API cache to disk. Do not treat 304 as an error code.
Add debugging logs.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 11 (+1416 -5) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 11) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Add dependancy on r/6548 so that
six.iteritems
can be used.Cache the Vary Headers correctly -- if none are sent, we need to record that none were sent.
Minor formatting and rename headers to vary_headers when referring specifically to the Vary header
Add testing info.
Testing Done: |
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Depends On: |
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 12 (+1414 -7) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/rbt/commands/clearcache.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 12) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Add note about unit tests passing.
Testing Done: |
|
---|
-
<p>Looking pretty awesome!</p>
<p>Caught some typos and style differences. The general logic looks pretty thorough and correct from what I can tell so far.</p> -
docs/rbtools/index.rst (Diff revision 12) These should each be using the
:file:
markup. (I know we don't have that above, but we should there as well.):file:`~/.cache/.rbtools/apicache.db`
-
docs/rbtools/rbt/commands/clearcache.rst (Diff revision 12) We should maybe say the HTTP cache for the API.
-
docs/rbtools/rbt/commands/index.rst (Diff revision 12) This and the doc page should match the command:
clear-cache
-
-
rbtools/api/cache.py (Diff revision 12) This isn't part of Python, so it must go in import group #2.
-
rbtools/api/cache.py (Diff revision 12) This is a Python standard library module, so it can go in the group above.
-
rbtools/api/cache.py (Diff revision 12) What's the reason for setting
__slots__
here?Generally speaking, there's rarely a reason to actually set this. It can be used for memory performance reasons, but unless we're expecting a large number of these to stick around in memory for a while, we probably don't gain enough to warrant here.
-
rbtools/api/cache.py (Diff revision 12) The function name and docstring sort of contradict each other.
-
rbtools/api/cache.py (Diff revision 12) Might be worth mentioning that this is intended to be API-compatible with a urllib2's response object. Same with
CachedResponse
below.Also, maybe we should call this
HTTPResponse
? AndCachedHTTPResponse
below? Just to make it more specific. (Not sure, just floating the idea.) -
-
rbtools/api/cache.py (Diff revision 12) Trailing period at the end of the log message.
Variable on the next line.
-
-
rbtools/api/cache.py (Diff revision 12) These could result in exceptions as well, right? Anything we should be catching here?
Also, you could skip doing this connect twice by doing:
cache_is_new = not os.path.exists(self.cache_path): if cache_is_new: # All the setup stuff for creating self.db = sqlite3.connect(...) if not cache_is_new: # Check version, etc.
-
-
rbtools/api/cache.py (Diff revision 12) Unless we're explicitly needing to worry about the difference between None, empty strings, 0, etc., it's usually best to just do
if entry:
.Same below.
-
-
rbtools/api/cache.py (Diff revision 12) In this particular case, it's fine to use
\
instead of parens for line continuation. -
-
-
-
rbtools/api/cache.py (Diff revision 12) We return
response
in all of these cases. We should probably just do a singlereturn response
at the end.The code above that returns a
CachedResponse
can just assign that toresponse
to take advantage of the single exit point. -
rbtools/api/cache.py (Diff revision 12) Must be on one line. Doc generators will chop it off otherwise.
-
rbtools/api/cache.py (Diff revision 12) I read this incorrectly at first. Can you add a small comment saying that we restore the old value below?
Also, this isn't thread-safe. Not a massive deal right now, but potentially problematic later. Easily fixed with doing all this in a
threading.Lock()
. -
-
-
rbtools/api/cache.py (Diff revision 12) How about "The C locale is not available on this system. The 'Expires' header cannot be parsed."
-
rbtools/api/cache.py (Diff revision 12) We should actually grab the old locale and set it, instead of calling
resetlocale
, in case the caller has actually set a custom locale before all this. -
-
-
rbtools/api/cache.py (Diff revision 12) Should be on the same line.
I'd rather not do the map/strip trick inline like this, and like the one above. Instead, can we make a utility function for parsing headr values, and use a list comprehension?
return [ s.strip() for s in headers.split(',') ]
-
rbtools/api/cache.py (Diff revision 12) We don't need to check against None unless there's a reason to worry about empty strings and zero. I believe those values are just as None-worthy as None for our purposes.
-
-
-
-
rbtools/api/cache.py (Diff revision 12) We use
dumps
to serialize, but are loading above with a custom split.dumps
may be too heavy. Can we just do a join? -
-
-
-
rbtools/commands/clearcache.py (Diff revision 12) How about "Deletes the HTTP cache used for the API."
-
rbtools/commands/clearcache.py (Diff revision 12) Rather than this, how about a method on
ApiCache
for deleting the cache? -
rbtools/commands/clearcache.py (Diff revision 12) Parameters should go on the line after the log message.
Change Summary:
Fix most issues outline in Christian's review.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 13 (+1429 -10) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 34.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 13) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
Change Summary:
Formatting. Fix
_split_csv
usage.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 14 (+1429 -10) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 14) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Catch exceptions around DB interaction and die gracefully. Rename
ApiCache
toAPICache
.
Description: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Testing Done: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Commit: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 15 (+1466 -10) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 15) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Use
six.moves
forurlopen
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 16 (+1466 -10) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 16) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
Looking great! Only a few small things left.
-
-
-
-
rbtools/api/cache.py (Diff revision 16) Instead of terminating the process, can we just not cache? RBTools is often run in places where it's locked down and would never be able to create a cache.
Change Summary:
Rebased off master. Fixed some of Christian's issues.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 17 (+1467 -11) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 17) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Make this review request depend on David's
Use six.moves for StringIO
review request.
Depends On: |
|
||
---|---|---|---|
Diff: |
Revision 18 (+1467 -11) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 18) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 19) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
Change Summary:
Add a try..except around the last DB operation.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 20 (+1477 -21) |
-
Tool: Pyflakes Processed Files: rbtools/utils/testbase.py rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/utils/testbase.py rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 20) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
A few things I didn't catch before. I don't see anything beyond these though. :)
-
rbtools/api/cache.py (Diff revision 20) For consistency with the RB database, mind if we use
cache_info
? -
-
-
-
-
-
-
-
Change Summary:
Rename
APICache
->api_cache
andCacheInfo
->cache_info
in database.Switch to using
assertTrue
/assertFalse
withisinstance
instead ofassertIsInstanceOf
/assertIsNotInstanceOf
in tests.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 21 (+1472 -12) |
-
Tool: Pyflakes Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst Tool: PEP8 Style Checker Processed Files: rbtools/api/tests.py rbtools/utils/appdirs.py rbtools/commands/clearcache.py rbtools/api/cache.py setup.py rbtools/api/request.py Ignored Files: rbtools/utils/COPYING.appdirs docs/rbtools/rbt/commands/clear-cache.rst rbtools/utils/README.appdirs docs/rbtools/rbt/commands/index.rst docs/rbtools/index.rst WARNING: Number of comments exceeded maximum, showing 30 of 32.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
rbtools/utils/appdirs.py (Diff revision 21) Col: 30 E128 continuation line under-indented for visual indent
-
-
-
-
-