Summary

Export/import review requests tool

Review Request #7952 — Created Feb. 8, 2016 and discarded Dec. 29, 2016, 10:31 a.m.

Information

Owner

xingc

Repository

Review Board

Branch

release-2.5.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

xingc

Description

Exported review requests are save into file in json format, and this file can be imported to recreate review requests. Objects to b serialized are put in topological order based on dependencies.

Testing Done

Manually export and import review requests with/without local site to see the results.
Export all non local site review requests to a file.

Export all local sites to a file.

Export a single local site to a file.

Import all local sites and recreate local sites from a file.

Import all local sites without local sites (merge into global) from a file.

Import all non local site review requests from a file and create a local site for them.
One assumption is that no IntegrityError, ie no Users with same username, a file is not imported twice, etc.

Import is atomic transaction therefore IntegrityError leads to roll back.

init.py and json.py are from Django's built-in serializers.

Issues

Description	From	Last Updated
Hm... I suspect this diff shouldn't be in here...	mike_conley	April 18, 2016, 9:34 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:34 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:33 a.m.
Col: 36 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:33 a.m.
Col: 49 E225 missing whitespace around operator	reviewbot	April 18, 2016, 9:34 a.m.
Col: 51 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:34 a.m.
Col: 53 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:34 a.m.
Col: 48 E225 missing whitespace around operator	reviewbot	April 18, 2016, 9:34 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:34 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:34 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:34 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:34 a.m.
Col: 80 E501 line too long (90 > 79 characters)	reviewbot	April 18, 2016, 9:34 a.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 18, 2016, 9:34 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
undefined name 'self'	reviewbot	April 18, 2016, 9:35 a.m.
undefined name 'self'	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:35 a.m.
Col: 46 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
Col: 18 E703 statement ends with a semicolon	reviewbot	April 18, 2016, 9:35 a.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (91 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (93 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:35 a.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (87 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (86 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 21 W291 trailing whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (88 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 88 E703 statement ends with a semicolon	reviewbot	April 18, 2016, 9:35 a.m.
Col: 9 E303 too many blank lines (2)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:35 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:35 a.m.
undefined name 'self'	reviewbot	April 18, 2016, 9:35 a.m.
undefined name 'self'	reviewbot	April 18, 2016, 9:35 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:35 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 29 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 31 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:35 a.m.
Col: 47 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 49 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 46 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 48 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 46 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:36 a.m.
Col: 18 E703 statement ends with a semicolon	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 20 E127 continuation line over-indented for visual indent	reviewbot	April 18, 2016, 9:36 a.m.
Col: 20 E127 continuation line over-indented for visual indent	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:36 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 29 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 31 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 47 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 49 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 46 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 48 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:36 a.m.
Col: 46 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:36 a.m.
Col: 18 E703 statement ends with a semicolon	reviewbot	April 18, 2016, 9:36 a.m.
Col: 62 W291 trailing whitespace	reviewbot	April 18, 2016, 9:36 a.m.
Col: 13 E113 unexpected indentation	reviewbot	April 18, 2016, 9:36 a.m.
Col: 69 W291 trailing whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 73 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:37 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 29 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 31 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 47 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 49 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:37 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:37 a.m.
Col: 5 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:37 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:37 a.m.
Col: 46 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 48 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 46 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:37 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:37 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:37 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:37 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:37 a.m.
Col: 36 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 38 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 29 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 31 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 47 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:37 a.m.
Col: 49 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:38 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:38 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:38 a.m.
Col: 5 E265 block comment should start with '# '	reviewbot	April 18, 2016, 9:38 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:38 a.m.
Col: 46 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:38 a.m.
Col: 48 E251 unexpected spaces around keyword / parameter equals	reviewbot	April 18, 2016, 9:38 a.m.
Col: 46 E502 the backslash is redundant between brackets	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:38 a.m.
Col: 5 E301 expected 1 blank line, found 0	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 W293 blank line contains whitespace	reviewbot	April 18, 2016, 9:38 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:38 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:38 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:38 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:38 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:38 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:55 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:45 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:45 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 18, 2016, 9:45 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:55 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:55 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:55 a.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 18, 2016, 9:55 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:55 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:46 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:47 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:47 a.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 18, 2016, 9:47 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:48 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:48 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:49 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:49 a.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	April 18, 2016, 9:50 a.m.
Col: 80 E501 line too long (83 > 79 characters)	reviewbot	April 18, 2016, 9:50 a.m.
Col: 80 E501 line too long (97 > 79 characters)	reviewbot	April 18, 2016, 9:50 a.m.
Col: 80 E501 line too long (102 > 79 characters)	reviewbot	April 18, 2016, 9:50 a.m.
Col: 80 E501 line too long (93 > 79 characters)	reviewbot	April 18, 2016, 9:50 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:51 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:51 a.m.
Col: 80 E501 line too long (87 > 79 characters)	reviewbot	April 18, 2016, 9:51 a.m.
Col: 80 E501 line too long (81 > 79 characters)	reviewbot	April 18, 2016, 9:52 a.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	April 18, 2016, 9:52 a.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 18, 2016, 9:52 a.m.
Col: 80 E501 line too long (82 > 79 characters)	reviewbot	April 18, 2016, 9:52 a.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	April 18, 2016, 9:52 a.m.
undefined name 'getvalue'	reviewbot	April 18, 2016, 9:53 a.m.
local variable 'db' is assigned to but never used	reviewbot	April 18, 2016, 9:53 a.m.
'python' imported but unused	reviewbot	April 18, 2016, 9:54 a.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 18, 2016, 9:54 a.m.
local variable 'string_data' is assigned to but never used	reviewbot	April 18, 2016, 9:55 a.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:03 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:02 p.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:02 p.m.
Col: 41 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:02 p.m.
Col: 17 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3:02 p.m.
Col: 25 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:02 p.m.
Col: 25 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:02 p.m.
Col: 21 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:02 p.m.
Col: 73 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:02 p.m.
Col: 67 W291 trailing whitespace	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 68 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3:02 p.m.
Col: 1 W391 blank line at end of file	reviewbot	April 29, 2016, 3:02 p.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 17 E116 unexpected indentation (comment)	reviewbot	April 29, 2016, 3:02 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 17 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:02 p.m.
Col: 17 E731 do not assign a lambda expression, use a def	reviewbot	April 29, 2016, 3:01 p.m.
local variable 'db' is assigned to but never used	reviewbot	April 29, 2016, 3:01 p.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 3:01 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:01 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:01 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:01 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:01 p.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:01 p.m.
Col: 41 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:01 p.m.
Col: 17 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3:01 p.m.
Col: 25 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:01 p.m.
Col: 25 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:01 p.m.
Col: 21 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:01 p.m.
Col: 73 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:01 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:01 p.m.
Col: 67 W291 trailing whitespace	reviewbot	April 29, 2016, 3:01 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 68 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3 p.m.
Col: 1 W391 blank line at end of file	reviewbot	April 29, 2016, 3 p.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	April 29, 2016, 3 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 17 E116 unexpected indentation (comment)	reviewbot	April 29, 2016, 3 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 17 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3 p.m.
Col: 17 E731 do not assign a lambda expression, use a def	reviewbot	April 29, 2016, 3 p.m.
local variable 'db' is assigned to but never used	reviewbot	April 29, 2016, 3 p.m.
'base' imported but unused	reviewbot	April 29, 2016, 3 p.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 3 p.m.
'from reviewboard.site.models import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 3 p.m.
'six' imported but unused	reviewbot	April 29, 2016, 3 p.m.
'transaction' imported but unused	reviewbot	April 29, 2016, 2:56 p.m.
'connections' imported but unused	reviewbot	April 29, 2016, 2:56 p.m.
'router' imported but unused	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 80 E501 line too long (89 > 79 characters)	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:56 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 2:57 p.m.
local variable 'db' is assigned to but never used	reviewbot	April 29, 2016, 2:57 p.m.
'IntegrityError' imported but unused	reviewbot	April 29, 2016, 2:57 p.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 2:57 p.m.
'from reviewboard.site.models import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 2:59 p.m.
'six' imported but unused	reviewbot	April 29, 2016, 3:11 p.m.
'transaction' imported but unused	reviewbot	April 29, 2016, 3:11 p.m.
'connections' imported but unused	reviewbot	April 29, 2016, 3:11 p.m.
'router' imported but unused	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 80 E501 line too long (89 > 79 characters)	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:11 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:12 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:12 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:12 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:12 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:12 p.m.
'DEFAULT_DB_ALIAS' imported but unused	reviewbot	April 29, 2016, 3:09 p.m.
'IntegrityError' imported but unused	reviewbot	April 29, 2016, 3:09 p.m.
'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 3:11 p.m.
'from reviewboard.site.models import *' used; unable to detect undefined names	reviewbot	April 29, 2016, 3:11 p.m.
'six' imported but unused	reviewbot	April 29, 2016, 3:26 p.m.
'router' imported but unused	reviewbot	April 29, 2016, 3:26 p.m.
'connections' imported but unused	reviewbot	April 29, 2016, 3:26 p.m.
'transaction' imported but unused	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 80 E501 line too long (89 > 79 characters)	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 E101 indentation contains mixed spaces and tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
Col: 1 W191 indentation contains tabs	reviewbot	April 29, 2016, 3:26 p.m.
undefined name 'models'	reviewbot	April 29, 2016, 3:26 p.m.
undefined name 'models'	reviewbot	April 29, 2016, 3:22 p.m.
undefined name 'models'	reviewbot	April 29, 2016, 3:22 p.m.
undefined name 'models'	reviewbot	April 29, 2016, 3:22 p.m.
undefined name 'transaction'	reviewbot	April 29, 2016, 3:22 p.m.
undefined name 'transaction'	reviewbot	April 29, 2016, 3:27 p.m.
undefined name 'transaction'	reviewbot	April 29, 2016, 3:27 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:28 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:28 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:28 p.m.
Col: 13 E203 whitespace before ':'	reviewbot	April 29, 2016, 3:28 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:28 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:29 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:29 p.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 29, 2016, 3:30 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:30 p.m.
Col: 41 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:30 p.m.
Col: 17 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3:31 p.m.
Col: 1 E302 expected 2 blank lines, found 1	reviewbot	April 29, 2016, 3:31 p.m.
Col: 5 E303 too many blank lines (2)	reviewbot	April 29, 2016, 3:32 p.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 29, 2016, 3:32 p.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	April 29, 2016, 3:32 p.m.
Col: 77 E226 missing whitespace around arithmetic operator	reviewbot	April 29, 2016, 3:33 p.m.
Col: 29 W292 no newline at end of file	reviewbot	April 29, 2016, 3:34 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:34 p.m.
Col: 1 E122 continuation line missing indentation or outdented	reviewbot	April 29, 2016, 3:34 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:34 p.m.
Col: 68 E502 the backslash is redundant between brackets	reviewbot	April 29, 2016, 3:35 p.m.
Col: 13 E128 continuation line under-indented for visual indent	reviewbot	April 29, 2016, 3:35 p.m.
Col: 1 W391 blank line at end of file	reviewbot	April 29, 2016, 3:35 p.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	April 29, 2016, 3:36 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:36 p.m.
Col: 9 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:36 p.m.
Col: 13 E265 block comment should start with '# '	reviewbot	April 29, 2016, 3:36 p.m.
Col: 17 E731 do not assign a lambda expression, use a def	reviewbot	April 29, 2016, 3:37 p.m.
Col: 17 E731 do not assign a lambda expression, use a def	reviewbot	April 29, 2016, 3:37 p.m.
Col: 80 E501 line too long (92 > 79 characters)	reviewbot	April 29, 2016, 4:50 p.m.

Description:

~		Testing Done:
	~	Serialize single rr in a .json file, and recreate rr from file.
-		commend: ./reviewboard/manage.py test -- testing.extensions
-
-		Reviewed at https://reviews.reviewboard.org/r/7915/
-
-		Merge branch 'master' of git://github.com/reviewboard/reviewboard
-
-
-
-		add export/import review requests tool

Testing Done:

Manually see the results.

Change Summary:

Tesing serialization and deserialization on a new django app

Added Files:

reviewboard/testing/extensions.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Hm... I suspect this diff shouldn't be in here...
```

Change Summary:

Add natural_key() to related classes, and get_by_natural_key() to related model managers

Branch:

-	master
+	release-2.5.x

Diff:

Revision 2 (+48 -4)

Show changes

	reviewboard/changedescs/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/site/models.py

Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py

reviewboard/changedescs/models.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 36
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 49
 E225 missing whitespace around operator
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 51
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 53
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/managers.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 48
 E225 missing whitespace around operator
```
reviewboard/reviews/models/group.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/tools.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (90 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/site/models.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/site/models.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```

Change Summary:

Added natural keys to DiffSetHistory, Repository, and Tool

Diff:

Revision 3 (+111 -7)

Show changes

	.reviewboardrc
	reviewboard/changedescs/models.py
	reviewboard/diffviewer/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/scmtools/managers.py
	2 more

Tool: Pyflakes
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/scmtools/models.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py

Ignored Files:
    .reviewboardrc



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/scmtools/models.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py

Ignored Files:
    .reviewboardrc


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

reviewboard/changedescs/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/changedescs/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
 undefined name 'self'
```
reviewboard/changedescs/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
 undefined name 'self'
```
reviewboard/diffviewer/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E265 block comment should start with '# '
```
reviewboard/reviews/managers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 46
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/models/group.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```

reviewboard/reviews/tools.py (Diff revision 3)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 18
 E703 statement ends with a semicolon
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (91 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (93 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (87 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (86 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 21
 W291 trailing whitespace
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (88 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 88
 E703 statement ends with a semicolon
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 9
 E303 too many blank lines (2)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/scmtools/managers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/scmtools/managers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/scmtools/models.py (Diff revision 3)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```

Change Summary:

Added diff (DiffSetHistory, DiffSet, FileDiff, RawFileDiffData, LegacyFileDiffData) and Group

Diff:

Revision 4 (+200 -9)

Show changes

	reviewboard/changedescs/models.py
	reviewboard/diffviewer/managers.py
	reviewboard/diffviewer/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/scmtools/managers.py
	2 more

Tool: Pyflakes
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/scmtools/models.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/diffviewer/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/scmtools/models.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/diffviewer/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py


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

reviewboard/changedescs/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/changedescs/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 undefined name 'self'
```
reviewboard/changedescs/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
 undefined name 'self'
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 29
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 31
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 47
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 49
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/diffviewer/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/models.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 5
 E265 block comment should start with '# '
```
reviewboard/reviews/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 46
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 48
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 46
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/models/group.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/group.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```

reviewboard/reviews/tools.py (Diff revision 4)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 18
 E703 statement ends with a semicolon
```
reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 20
 E127 continuation line over-indented for visual indent
```
reviewboard/reviews/tools.py (Diff revision 4)
The issue has been dropped. Show all issues
```
Col: 20
 E127 continuation line over-indented for visual indent
```

Change Summary:

Fixed old objects being overridden problem. Tyring ways to make unique natural_key().

Diff:

Revision 5 (+202 -9)

Show changes

	reviewboard/changedescs/models.py
	reviewboard/diffviewer/managers.py
	reviewboard/diffviewer/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/scmtools/managers.py
	2 more

Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/scmtools/models.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/diffviewer/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py


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

reviewboard/changedescs/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 29
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 31
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 47
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 49
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/diffviewer/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/models.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 5
 E265 block comment should start with '# '
```
reviewboard/reviews/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 46
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 48
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 46
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/models/group.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/group.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 18
 E703 statement ends with a semicolon
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 62
 W291 trailing whitespace
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 13
 E113 unexpected indentation
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 69
 W291 trailing whitespace
```
reviewboard/reviews/tools.py (Diff revision 5)
The issue has been dropped. Show all issues
```
Col: 73
 E502 the backslash is redundant between brackets
```

Change Summary:

Modified Django's built-in Serializers code to use uuid

Diff:

Revision 6 (+790 -9)

Show changes

	reviewboard/changedescs/models.py
	reviewboard/diffviewer/managers.py
	reviewboard/diffviewer/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/reviews/serializers/__init__.py
	6 more

Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/reviews/serializers/json.py
    reviewboard/scmtools/models.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/reviews/serializers/base.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/diffviewer/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py


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

reviewboard/changedescs/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 29
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 31
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 47
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 49
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/diffviewer/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/models.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 5
 E265 block comment should start with '# '
```
reviewboard/reviews/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 46
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 48
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 46
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/models/group.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/group.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 6)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```

Change Summary:

Changed Django deserializer code

Diff:

Revision 7 (+820 -9)

Show changes

	reviewboard/changedescs/models.py
	reviewboard/diffviewer/managers.py
	reviewboard/diffviewer/models.py
	reviewboard/reviews/managers.py
	reviewboard/reviews/tools.py
	reviewboard/reviews/models/group.py
	reviewboard/reviews/models/review_request.py
	reviewboard/reviews/serializers/__init__.py
	6 more

Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/managers.py
    reviewboard/reviews/serializers/json.py
    reviewboard/scmtools/models.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/diffviewer/models.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/models/group.py
    reviewboard/reviews/models/review_request.py
    reviewboard/reviews/serializers/base.py
    reviewboard/site/models.py
    reviewboard/reviews/managers.py
    reviewboard/diffviewer/managers.py
    reviewboard/reviews/tools.py
    reviewboard/changedescs/models.py


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

reviewboard/changedescs/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 36
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 38
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 29
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 31
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 47
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 49
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/diffviewer/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/diffviewer/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/diffviewer/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/diffviewer/models.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 5
 E265 block comment should start with '# '
```
reviewboard/reviews/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 46
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 48
 E251 unexpected spaces around keyword / parameter equals
```
reviewboard/reviews/managers.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 46
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/models/group.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/group.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/models/review_request.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 5
 E301 expected 1 blank line, found 0
```
reviewboard/reviews/models/review_request.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 W293 blank line contains whitespace
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 7)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```

There are a lot of old open issues on old reviews (mostly from Review Bot). Please mark these as "Fixed" so we have an accurate count of which things still have yet to be addressed.

Removed Files:

Change Summary:

Removed previous unnecessary changes

Diff:

Revision 8 (+751)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py


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

reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (83 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (97 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (102 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (93 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (87 > 79 characters)
```
reviewboard/reviews/serializers/json.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (81 > 79 characters)
```
reviewboard/reviews/serializers/json.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (85 > 79 characters)
```
reviewboard/reviews/serializers/json.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/json.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (82 > 79 characters)
```
reviewboard/reviews/serializers/json.py (Diff revision 8)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (85 > 79 characters)
```
reviewboard/reviews/serializers/python.py (Diff revision 8)
The issue has been resolved. Show all issues
```
 undefined name 'getvalue'
```
reviewboard/reviews/serializers/python.py (Diff revision 8)
The issue has been dropped. Show all issues
```
 local variable 'db' is assigned to but never used
```
reviewboard/reviews/tools.py (Diff revision 8)
The issue has been resolved. Show all issues
```
 'python' imported but unused
```

reviewboard/reviews/tools.py (Diff revision 8)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 8)
The issue has been resolved. Show all issues
```
 local variable 'string_data' is assigned to but never used
```

Change Summary:

Fixed style

Diff:

Revision 9 (+768)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py


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

reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 41
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/__init__.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 17
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/base.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 25
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 25
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 21
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 73
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 67
 W291 trailing whitespace
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 68
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/json.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (85 > 79 characters)
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 17
 E116 unexpected indentation (comment)
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 17
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
Col: 17
 E731 do not assign a lambda expression, use a def
```
reviewboard/reviews/serializers/python.py (Diff revision 9)
The issue has been dropped. Show all issues
```
 local variable 'db' is assigned to but never used
```

reviewboard/reviews/tools.py (Diff revision 9)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

Change Summary:

Added more use cases

Diff:

Revision 10 (+833)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py


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

reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 41
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/__init__.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 17
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/base.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 25
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 25
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 21
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/base.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 73
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 67
 W291 trailing whitespace
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 68
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/json.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (85 > 79 characters)
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 17
 E116 unexpected indentation (comment)
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 17
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
Col: 17
 E731 do not assign a lambda expression, use a def
```
reviewboard/reviews/serializers/python.py (Diff revision 10)
The issue has been dropped. Show all issues
```
 local variable 'db' is assigned to but never used
```
reviewboard/reviews/tools.py (Diff revision 10)
The issue has been dropped. Show all issues
```
 'base' imported but unused
```

reviewboard/reviews/tools.py (Diff revision 10)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 10)

The issue has been dropped. Show all issues

 'from reviewboard.site.models import *' used; unable to detect undefined names

Change Summary:

fixed review requests not showing in newly created local site

Diff:

Revision 11 (+797)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/RBBase.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py


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

reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 'six' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 'transaction' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 'connections' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 'router' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (89 > 79 characters)
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 11)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/python.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 local variable 'db' is assigned to but never used
```
reviewboard/reviews/tools.py (Diff revision 11)
The issue has been dropped. Show all issues
```
 'IntegrityError' imported but unused
```

reviewboard/reviews/tools.py (Diff revision 11)

The issue has been dropped. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 11)

The issue has been dropped. Show all issues

 'from reviewboard.site.models import *' used; unable to detect undefined names

Change Summary:

Removed unnecessary file

Diff:

Revision 12 (+796)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/RBBase.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py


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

reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
 'six' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
 'transaction' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
 'connections' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
 'router' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (89 > 79 characters)
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 12)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/python.py (Diff revision 12)
The issue has been resolved. Show all issues
```
 'DEFAULT_DB_ALIAS' imported but unused
```
reviewboard/reviews/tools.py (Diff revision 12)
The issue has been resolved. Show all issues
```
 'IntegrityError' imported but unused
```

reviewboard/reviews/tools.py (Diff revision 12)

The issue has been resolved. Show all issues

 'from reviewboard.reviews.models.review_request import *' used; unable to detect undefined names

reviewboard/reviews/tools.py (Diff revision 12)

The issue has been resolved. Show all issues

 'from reviewboard.site.models import *' used; unable to detect undefined names

Change Summary:

Improved style

Diff:

Revision 13 (+794)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/RBBase.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/serializers/__init__.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/RBBase.py
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/tools.py


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

reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
 'six' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
 'router' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
 'connections' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
 'transaction' imported but unused
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (89 > 79 characters)
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 E101 indentation contains mixed spaces and tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/RBBase.py (Diff revision 13)
The issue has been dropped. Show all issues
```
Col: 1
 W191 indentation contains tabs
```
reviewboard/reviews/serializers/python.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'models'
```
reviewboard/reviews/serializers/python.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'models'
```
reviewboard/reviews/serializers/python.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'models'
```
reviewboard/reviews/serializers/python.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'models'
```
reviewboard/reviews/tools.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'transaction'
```
reviewboard/reviews/tools.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'transaction'
```
reviewboard/reviews/tools.py (Diff revision 13)
The issue has been resolved. Show all issues
```
 undefined name 'transaction'
```

Change Summary:

Improved style

Diff:

Revision 14 (+744)

Show changes

	reviewboard/reviews/tools.py
	reviewboard/reviews/serializers/__init__.py
	reviewboard/reviews/serializers/base.py
	reviewboard/reviews/serializers/json.py
	reviewboard/reviews/serializers/python.py

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/serializers/base.py
    reviewboard/reviews/serializers/python.py
    reviewboard/reviews/serializers/json.py
    reviewboard/reviews/tools.py
    reviewboard/reviews/serializers/__init__.py


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

reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 13
 E203 whitespace before ':'
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 41
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/__init__.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 17
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E302 expected 2 blank lines, found 1
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 5
 E303 too many blank lines (2)
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 77
 E226 missing whitespace around arithmetic operator
```
reviewboard/reviews/serializers/base.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 29
 W292 no newline at end of file
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 E122 continuation line missing indentation or outdented
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 68
 E502 the backslash is redundant between brackets
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 13
 E128 continuation line under-indented for visual indent
```
reviewboard/reviews/serializers/json.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 1
 W391 blank line at end of file
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (85 > 79 characters)
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 9
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been resolved. Show all issues
```
Col: 13
 E265 block comment should start with '# '
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 17
 E731 do not assign a lambda expression, use a def
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 17
 E731 do not assign a lambda expression, use a def
```
reviewboard/reviews/serializers/python.py (Diff revision 14)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (92 > 79 characters)
```

Change Summary:

Updated description and test messages

Description:

~		Serialize single rr in a .json file, and recreate rr from file.
	~	Exported review requests are save into file in json format, and this file can be imported to recreate review requests. Objects to b serialized are put in topological order based on dependencies.

Testing Done:

~		Manually see the results.
	~	Manually export and import review requests with/without local site to see the results.
	+
	+	Export all non local site review requests to a file.
	+	Export all local sites to a file.
	+	Export a single local site to a file.
	+	Import all local sites and recreate local sites from a file.
	+	Import all local sites without local sites (merge into global) from a file.
	+	Import all non local site review requests from a file and create a local site for them.
	+
	+	One assumption is that no IntegrityError, ie no Users with same username, a file is not imported twice, etc.
	+	Import is atomic transaction therefore IntegrityError leads to roll back.
	+	init.py and json.py are from Django's built-in serializers.

People:

+	xingc

Change Summary:

Removed [WIP]

Summary:

-	[WIP]add export/import review requests tool
+	Export/import review requests tool

Export/import review requests tool

Information

Reviewers

Description:

Testing Done:

Change Summary:

Added Files:

Change Summary:

Branch:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Removed Files:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Diff:

Change Summary:

Description:

Testing Done:

People:

Change Summary:

Summary:

Status: Discarded