Summary

Status API

Review Request #7961 — Created Feb. 14, 2016 and discarded Sept. 16, 2016, 5:03 p.m.

Information

Owner

shuaisha

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard, students

People

Description

Basic GET/POST/PUT request handlers are done. The basic statuses has type Success(represented as "S"), Failure("F") and Pending("P").
Unit tests are written for new features.
UI for displaying statuses are created.

A demo for latest version can be viewed at:
https://drive.google.com/file/d/0B9f_D8xqWtODX01NMTFiZ2hfOG8/view?usp=sharing

What to be done next:
Statuses are referenced by internally generated status id. Should change to a unique client ID.
Work with ReviewBot on grouping reviews of latest statuses.
Add filter to whether the statuses are current for the latest change (should probably default to only showing for the latest change)

It's best to start looking at webapi/resources/status.py and reviews/models/status.py for implementation.

Some sample requests to be made are:
Get all statuses on a review request:
curl host/api/review-requests/1/statuss/

GET request with some filter options:
curl host/api/review-requests/1/statuss/?status_type=P
curl host/api/review-requests/1/statuss/?time=YYYY-MM-DD-HH-MM-SS

POST request (need authorization info):
curl -X POST host/api/review-requests/1/statuss/ -d "context=Travis%20CI&label=Failed&progress=F&url=continuous-integration/travis&url_label=Build%20log" -A application/json

PUT request (need authorization info):
curl -X PUT host/api/review-requests/1/statuss/1/ -d "context=Unit%20Testing&label=Success&progress=S" -A application/json

Testing Done

Issues

Description	From	Last Updated
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 63 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 60 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:48 p.m.
Col: 76 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 33 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 62 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 1 W391 blank line at end of file	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 54 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 56 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 64 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 18 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 20 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 53 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 24 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 26 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 56 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 58 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 74 W291 trailing whitespace	reviewbot	Feb. 21, 2016, 7:49 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	Feb. 28, 2016, 8:20 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:20 p.m.
Col: 59 W291 trailing whitespace	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 63 W291 trailing whitespace	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 76 W291 trailing whitespace	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 33 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 62 W291 trailing whitespace	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 1 W391 blank line at end of file	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 58 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 60 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 76 W291 trailing whitespace	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 17 E128 continuation line under-indented for visual indent	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 22 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 24 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 65 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 67 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 26 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
Col: 28 E251 unexpected spaces around keyword / parameter equals	reviewbot	Feb. 28, 2016, 8:19 p.m.
'ObjectDoesNotExist' imported but unused	reviewbot	Feb. 28, 2016, 8:19 p.m.
'Q' imported but unused	reviewbot	Feb. 28, 2016, 8:20 p.m.
'webapi_response_errors' imported but unused	reviewbot	Feb. 28, 2016, 8:20 p.m.
'webapi_request_fields' imported but unused	reviewbot	Feb. 28, 2016, 8:20 p.m.
'webapi_login_required' imported but unused	reviewbot	Feb. 28, 2016, 8:20 p.m.
'resources' imported but unused	reviewbot	Feb. 28, 2016, 8:20 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	March 21, 2016, 12:41 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:40 a.m.
Col: 59 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	March 21, 2016, 12:42 a.m.
Col: 63 W291 trailing whitespace	reviewbot	March 21, 2016, 12:42 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 76 W291 trailing whitespace	reviewbot	March 21, 2016, 12:42 a.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 33 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 62 W291 trailing whitespace	reviewbot	March 21, 2016, 12:42 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 1 W391 blank line at end of file	reviewbot	March 21, 2016, 12:42 a.m.
Col: 58 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 60 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 76 W291 trailing whitespace	reviewbot	March 21, 2016, 12:42 a.m.
Col: 17 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:42 a.m.
Col: 22 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 24 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 65 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 67 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 26 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
Col: 28 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:42 a.m.
'webapi_check_local_site' imported but unused	reviewbot	March 21, 2016, 12:42 a.m.
undefined name 'review'	reviewbot	March 21, 2016, 12:42 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	March 21, 2016, 12:41 a.m.
Col: 63 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 76 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 33 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 62 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
Col: 1 W391 blank line at end of file	reviewbot	March 21, 2016, 12:41 a.m.
'randint' imported but unused	reviewbot	March 21, 2016, 12:41 a.m.
Col: 78 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	March 21, 2016, 12:41 a.m.
Col: 80 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 77 W291 trailing whitespace	reviewbot	March 21, 2016, 12:41 a.m.
Col: 41 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:41 a.m.
Col: 39 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:41 a.m.
Col: 25 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:41 a.m.
'webapi_check_local_site' imported but unused	reviewbot	March 21, 2016, 12:41 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	March 21, 2016, 12:41 a.m.
Col: 48 W291 trailing whitespace	reviewbot	March 21, 2016, 12:40 a.m.
Col: 71 E502 the backslash is redundant between brackets	reviewbot	March 21, 2016, 12:40 a.m.
Col: 71 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:40 a.m.
Col: 21 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:40 a.m.
Col: 28 E131 continuation line unaligned for hanging indent	reviewbot	March 21, 2016, 12:40 a.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:40 a.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	March 21, 2016, 12:40 a.m.
Col: 59 E251 unexpected spaces around keyword / parameter equals	reviewbot	March 21, 2016, 12:40 a.m.
undefined name 'PublishError'	reviewbot	March 21, 2016, 12:40 a.m.
'mail' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'add_fixtures' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'PERMISSION_DENIED' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'DOES_NOT_EXIST' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'NOT_LOGGED_IN' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'ReviewRequestChildItemMixin' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
'get_status_item_url' imported but unused	reviewbot	March 21, 2016, 12:40 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 11, 2016, 12:09 p.m.
Col: 63 W291 trailing whitespace	reviewbot	April 11, 2016, 12:09 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 9 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 76 W291 trailing whitespace	reviewbot	April 11, 2016, 12:13 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 33 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 62 W291 trailing whitespace	reviewbot	April 11, 2016, 12:13 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:13 p.m.
Col: 37 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:14 p.m.
Col: 29 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:14 p.m.
Col: 1 W391 blank line at end of file	reviewbot	April 11, 2016, 12:14 p.m.
'randint' imported but unused	reviewbot	April 11, 2016, 12:14 p.m.
Col: 78 W291 trailing whitespace	reviewbot	April 11, 2016, 12:14 p.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	April 11, 2016, 12:15 p.m.
Col: 80 W291 trailing whitespace	reviewbot	April 11, 2016, 12:15 p.m.
Col: 77 W291 trailing whitespace	reviewbot	April 11, 2016, 12:15 p.m.
Col: 25 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:15 p.m.
Col: 39 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 11, 2016, 12:16 p.m.
Col: 41 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 11, 2016, 12:16 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 11, 2016, 12:17 p.m.
Col: 48 W291 trailing whitespace	reviewbot	April 11, 2016, 12:18 p.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 11, 2016, 12:18 p.m.
Col: 71 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 11, 2016, 12:20 p.m.
Col: 71 E502 the backslash is redundant between brackets	reviewbot	April 11, 2016, 12:20 p.m.
Col: 21 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:20 p.m.
Col: 28 E131 continuation line unaligned for hanging indent	reviewbot	April 11, 2016, 12:21 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:22 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:22 p.m.
undefined name 'PublishError'	reviewbot	April 11, 2016, 12:22 p.m.
'mail' imported but unused	reviewbot	April 11, 2016, 12:22 p.m.
'add_fixtures' imported but unused	reviewbot	April 11, 2016, 12:22 p.m.
'NOT_LOGGED_IN' imported but unused	reviewbot	April 11, 2016, 12:23 p.m.
'DOES_NOT_EXIST' imported but unused	reviewbot	April 11, 2016, 12:23 p.m.
'ChangeDescription' imported but unused	reviewbot	April 11, 2016, 12:23 p.m.
'PUBLISH_ERROR' imported but unused	reviewbot	April 11, 2016, 12:28 p.m.
Col: 28 E131 continuation line unaligned for hanging indent	reviewbot	April 11, 2016, 12:29 p.m.
Col: 80 E501 line too long (86 > 79 characters)	reviewbot	April 11, 2016, 12:30 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 11, 2016, 12:31 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 11, 2016, 12:31 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:31 p.m.
Col: 65 W291 trailing whitespace	reviewbot	April 11, 2016, 12:32 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:33 p.m.
Col: 13 E123 closing bracket does not match indentation of opening bracket's line	reviewbot	April 11, 2016, 12:33 p.m.
Col: 65 W291 trailing whitespace	reviewbot	April 11, 2016, 12:34 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:34 p.m.
Col: 13 E123 closing bracket does not match indentation of opening bracket's line	reviewbot	April 11, 2016, 12:34 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 11, 2016, 12:34 p.m.
Col: 72 W291 trailing whitespace	reviewbot	April 11, 2016, 12:34 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:35 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 11, 2016, 12:38 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:38 p.m.
Col: 75 W291 trailing whitespace	reviewbot	April 11, 2016, 12:39 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 11, 2016, 12:39 p.m.
Col: 13 E123 closing bracket does not match indentation of opening bracket's line	reviewbot	April 11, 2016, 12:40 p.m.
Col: 1 W391 blank line at end of file	reviewbot	April 11, 2016, 12:40 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 11, 2016, 12:40 p.m.
Col: 68 E502 the backslash is redundant between brackets	reviewbot	April 25, 2016, 1:37 p.m.
Col: 68 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 25, 2016, 1:50 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 25, 2016, 1:50 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 25, 2016, 1:50 p.m.
Col: 68 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 25, 2016, 1:53 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 25, 2016, 1:53 p.m.
Col: 17 E126 continuation line over-indented for hanging indent	reviewbot	April 25, 2016, 1:55 p.m.
Since this is a status, do we really need "status label", "status progress" and "status description"? I imagine it should …	mike_conley	April 29, 2016, 6:08 p.m.
We tend to use underscores in our variable names to break up multiple words. This should probably be num_failures. Also, …	mike_conley	April 29, 2016, 6:13 p.m.
Maybe use count() here instead, unless there's a good reason not to (the only reason we might want len(), as …	mike_conley	April 29, 2016, 6:16 p.m.
Nit - one space indentation for the HTML in templates. Same goes for the rest of your templates.	mike_conley	April 29, 2016, 6:22 p.m.
If we can, we should probably avoid inline styling and put this in the CSS for review details instead.	mike_conley	April 29, 2016, 6:39 p.m.
This needs to be a translatable string - see the "Change Summary:" translation being done on line 42.	mike_conley	April 29, 2016, 7:19 p.m.
Same as above, re: inline styling. We can use either a class or an attribute to set the state, and …	mike_conley	April 29, 2016, 6:53 p.m.
Please don't use inline styling if it can be avoided.	mike_conley	April 29, 2016, 6:53 p.m.
Translate this too, please.	mike_conley	April 29, 2016, 7:19 p.m.
These <br>'s can probably be avoided here and on line 44 with some margin CSS.	mike_conley	April 29, 2016, 6:54 p.m.
More inline styles to switch to CSS.	mike_conley	April 29, 2016, 6:54 p.m.
Can you add a comment about why we're skipping rendering this if there's at least one ChangeDescription? Is this because …	mike_conley	April 29, 2016, 7:10 p.m.
I don't think this is going out in 2.6. This should probably be 3.0 or 4.0. Maybe ask Christian.	mike_conley	April 29, 2016, 6:56 p.m.
Col: 80 E501 line too long (83 > 79 characters)	reviewbot	April 29, 2016, 8:31 p.m.
Col: 65 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 29, 2016, 8:33 p.m.
Col: 65 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 8:33 p.m.
Col: 65 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 29, 2016, 8:35 p.m.

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/reviews/models/status.py
    reviewboard/reviews/models/__init__.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/reviews/models/status.py
    reviewboard/reviews/models/__init__.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html


WARNING: Number of comments exceeded maximum, showing 30 of 33.

reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 63
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 60
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 33
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 54
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 56
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 64
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 18
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 20
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 53
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 24
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 26
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 56
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 58
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 74
 W291 trailing whitespace
```

Description:

~		Try adding status into UI
	~	Try adding status into UI;
	+	Start creating status API web resource.

Diff:

Revision 2 (+240 -2)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/evolutions/__init__.py
	reviewboard/reviews/evolutions/status_url_label.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/boxes/review.html
	reviewboard/webapi/resources/__init__.py
	reviewboard/webapi/resources/review_request.py
	1 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/reviews/evolutions/status_url_label.py
    reviewboard/reviews/evolutions/__init__.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/reviews/evolutions/status_url_label.py
    reviewboard/reviews/evolutions/__init__.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html


WARNING: Number of comments exceeded maximum, showing 30 of 42.

reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 59
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 63
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 33
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 58
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 60
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 17
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 22
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 24
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 65
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 67
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 26
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 28
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'ObjectDoesNotExist' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'Q' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'webapi_response_errors' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'webapi_request_fields' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'webapi_login_required' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 2)
The issue has been dropped. Show all issues
```
 'resources' imported but unused
```

Description:

~		Try adding status into UI;
	~	Added handler for GET request.
-		Start creating status API web resource.

Diff:

Revision 3 (+334 -2)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/evolutions/__init__.py
	reviewboard/reviews/evolutions/status_url_label.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/boxes/review.html
	reviewboard/webapi/resources/__init__.py
	reviewboard/webapi/resources/review_request.py
	1 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/reviews/evolutions/status_url_label.py
    reviewboard/reviews/evolutions/__init__.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/reviews/evolutions/status_url_label.py
    reviewboard/reviews/evolutions/__init__.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html


WARNING: Number of comments exceeded maximum, showing 30 of 44.

reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 59
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 63
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 33
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 58
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 60
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 17
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 22
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 24
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 65
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 67
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 26
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 28
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
 'webapi_check_local_site' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 3)
The issue has been dropped. Show all issues
```
 undefined name 'review'
```

Description:

~		Added handler for GET request.
	~	Implemented request handlers and added simple test cases

Diff:

Revision 4 (+541 -3)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/boxes/review.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	reviewboard/webapi/resources/review_request.py
	reviewboard/webapi/resources/status.py
	3 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html


WARNING: Number of comments exceeded maximum, showing 30 of 38.

reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 63
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 33
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'randint' imported but unused
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 78
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 80
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 77
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 41
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 39
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 25
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'webapi_check_local_site' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 48
 W291 trailing whitespace
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 71
 E502 the backslash is redundant between brackets
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 71
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 21
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 28
 E131 continuation line unaligned for hanging indent
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 59
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 undefined name 'PublishError'
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'mail' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'add_fixtures' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'PERMISSION_DENIED' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'DOES_NOT_EXIST' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'NOT_LOGGED_IN' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'ReviewRequestChildItemMixin' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 'get_status_item_url' imported but unused
```

Diff:

Revision 5 (+708 -3)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/boxes/review.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	reviewboard/webapi/resources/review_request.py
	reviewboard/webapi/resources/status.py
	3 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html


WARNING: Number of comments exceeded maximum, showing 30 of 52.

reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 63
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 9
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 76
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 33
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 37
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 29
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/models/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'randint' imported but unused
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 78
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 80
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 77
 W291 trailing whitespace
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 25
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 39
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/views.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 41
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 48
 W291 trailing whitespace
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 71
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 71
 E502 the backslash is redundant between brackets
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 21
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 28
 E131 continuation line unaligned for hanging indent
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/resources/status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 undefined name 'PublishError'
```
reviewboard/webapi/tests/test_status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'mail' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'add_fixtures' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'NOT_LOGGED_IN' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'DOES_NOT_EXIST' imported but unused
```
reviewboard/webapi/tests/test_status.py (Diff revision 5)
The issue has been resolved. Show all issues
```
 'ChangeDescription' imported but unused
```

Description:

~		Implemented request handlers and added simple test cases
	~	Fixed filter bug and added filter features.
	+	Fixed relation between Review and Status, and corresponding creations/update.

Diff:

Revision 6 (+736 -4)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/boxes/review.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	reviewboard/webapi/resources/base_review.py
	reviewboard/webapi/resources/review_request.py
	4 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/boxes/review.html

reviewboard/webapi/resources/status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
 'PUBLISH_ERROR' imported but unused
```
reviewboard/webapi/resources/status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 28
 E131 continuation line unaligned for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (86 > 79 characters)
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 65
 W291 trailing whitespace
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E123 closing bracket does not match indentation of opening bracket's line
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 65
 W291 trailing whitespace
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E123 closing bracket does not match indentation of opening bracket's line
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 72
 W291 trailing whitespace
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 75
 W291 trailing whitespace
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 13
 E123 closing bracket does not match indentation of opening bracket's line
```
reviewboard/webapi/tests/test_status.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/webapi/tests/urls.py (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```

Summary:

-	[WIP] Status API
+	Status API

Description:

~		Fixed filter bug and added filter features.
~		Fixed relation between Review and Status, and corresponding creations/update.
	~	Basic GET/POST/PUT request handlers are done.
	~	Unit tests are written for new features.
	+	Basic UI is created.

Diff:

Revision 7 (+799 -4)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/review_detail.html
	reviewboard/templates/reviews/boxes/change.html
	reviewboard/templates/reviews/boxes/status.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	6 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html

Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html

reviewboard/reviews/views.py (Diff revision 7)
The issue has been resolved. Show all issues
```
Col: 68
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/views.py (Diff revision 7)
The issue has been resolved. Show all issues
```
Col: 68
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/tests/test_status.py (Diff revision 7)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 7)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```

Diff:

Revision 8 (+794 -4)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/review_detail.html
	reviewboard/templates/reviews/boxes/change.html
	reviewboard/templates/reviews/boxes/status.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	6 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html

reviewboard/reviews/views.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 68
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/webapi/tests/test_status.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```
reviewboard/webapi/tests/test_status.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 17
 E126 continuation line over-indented for hanging indent
```

Diff:

Revision 9 (+791 -4)

Show changes

	reviewboard/reviews/views.py
	reviewboard/reviews/models/__init__.py
	reviewboard/reviews/models/status.py
	reviewboard/templates/reviews/review_detail.html
	reviewboard/templates/reviews/boxes/change.html
	reviewboard/templates/reviews/boxes/status.html
	reviewboard/testing/testcase.py
	reviewboard/webapi/resources/__init__.py
	6 more

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/views.py
    reviewboard/testing/testcase.py
    reviewboard/webapi/resources/__init__.py
    reviewboard/webapi/tests/test_status.py
    reviewboard/reviews/models/__init__.py
    reviewboard/webapi/resources/review_request.py
    reviewboard/webapi/resources/status.py
    reviewboard/reviews/models/status.py
    reviewboard/webapi/resources/base_review.py
    reviewboard/webapi/tests/mimetypes.py
    reviewboard/webapi/tests/urls.py

Ignored Files:
    reviewboard/templates/reviews/review_detail.html
    reviewboard/templates/reviews/boxes/status.html
    reviewboard/templates/reviews/boxes/change.html