Summary:

Create an Extension that suggests reviewers

Review Request #10248 — Created Oct. 20, 2018 and discarded March 8, 2019, 5:53 p.m.

Information
Owner:	praiseA
Repository:	Review Board
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	reviewboard, students
People:

Description

This extension would suggest reviewers for a current review request. It keeps
track of all previous reviews and suggests a username of someone who has
reviewed something similar in the past. The extension uses an ML model called
kNN algorithm by calculating the distance between past reviews and the current
review request.

Testing Done


Ran the ML model with past reviews and compared suggested reviewers with the 

actual person that reviewed the change set using SuggestReviewersAccuracy.py.
Settled on the combination of weights, and which variables to include in the 

distance calculation based on the combination that gave the highest accuracy score.

Please take a look at allresults.csv.

Issues

Description	From	Last Updated
Hey Praise! I looked at your code, and even after reading the Review Board documentation for extensions I still don't …	skaefer143	Nov. 19, 2018, 10:24 a.m.
E401 multiple imports on one line	reviewbot	Oct. 24, 2018, 9:01 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:02 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:02 p.m.
W601 .has_key() is deprecated, use 'in'	reviewbot	Oct. 24, 2018, 9:02 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:03 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:03 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:03 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:08 p.m.
E501 line too long (88 > 79 characters)	reviewbot	Oct. 24, 2018, 9:08 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 24, 2018, 9:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:08 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:08 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 9:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:08 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 9:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:08 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:08 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 9:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 9:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:10 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 9:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:09 p.m.
E501 line too long (102 > 79 characters)	reviewbot	Oct. 24, 2018, 9:09 p.m.
E501 line too long (117 > 79 characters)	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 9:09 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 9:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:24 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:24 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:24 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 9:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:24 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 9:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 9:24 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:24 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:24 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:37 p.m.
F401 'reviewboard.urls.diffviewer_url_names' imported but unused	reviewbot	Oct. 24, 2018, 9:37 p.m.
E303 too many blank lines (3)	reviewbot	Oct. 24, 2018, 9:37 p.m.
E302 expected 2 blank lines, found 0	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:37 p.m.
F401 'reviewboard.urls.diffviewer_url_names' imported but unused	reviewbot	Oct. 24, 2018, 9:37 p.m.
E303 too many blank lines (3)	reviewbot	Oct. 24, 2018, 9:37 p.m.
E302 expected 2 blank lines, found 0	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:37 p.m.
E501 line too long (92 > 79 characters)	reviewbot	Oct. 24, 2018, 9:37 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 9:37 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 9:37 p.m.
F401 'djblets.util.fields.JSONField' imported but unused	reviewbot	Oct. 24, 2018, 9:38 p.m.
F401 'reviewboard.reviews.models.ReviewRequest' imported but unused	reviewbot	Oct. 24, 2018, 9:38 p.m.
F401 'reviewboard.reviews.models.Review' imported but unused	reviewbot	Oct. 24, 2018, 9:38 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 9:38 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 9:38 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Oct. 24, 2018, 9:38 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 9:38 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 9:38 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 9:46 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 9:40 p.m.
E301 expected 1 blank line, found 0	reviewbot	Oct. 24, 2018, 9:46 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:46 p.m.
E225 missing whitespace around operator	reviewbot	Oct. 24, 2018, 9:46 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 9:46 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:47 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 9:47 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 9:47 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Oct. 24, 2018, 9:47 p.m.
F821 undefined name 'models'	reviewbot	Oct. 24, 2018, 9:47 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 9:47 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 9:50 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 9:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 24, 2018, 9:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 24, 2018, 9:50 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 8, 2018, 5:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:15 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 5:15 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:15 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 5:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:15 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:15 p.m.
E703 statement ends with a semicolon	reviewbot	Nov. 8, 2018, 5:15 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:16 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:16 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 5:16 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 5:16 p.m.
F821 undefined name 'calculateDistanceBetweenTwoDirs'	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
E501 line too long (83 > 79 characters)	reviewbot	Nov. 8, 2018, 5:16 p.m.
F821 undefined name 'dist'	reviewbot	Nov. 8, 2018, 5:16 p.m.
F821 undefined name 'operator'	reviewbot	Nov. 8, 2018, 5:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:17 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 8, 2018, 5:17 p.m.
E116 unexpected indentation (comment)	reviewbot	Nov. 8, 2018, 5:17 p.m.
F841 local variable 'review_request' is assigned to but never used	reviewbot	Nov. 8, 2018, 5:17 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Nov. 8, 2018, 5:17 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:17 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:18 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 5:18 p.m.
E501 line too long (91 > 79 characters)	reviewbot	Nov. 8, 2018, 5:18 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 5:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:18 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 5:18 p.m.
W391 blank line at end of file	reviewbot	Nov. 8, 2018, 5:18 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 5:18 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:18 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 5:18 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:18 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:19 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:19 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 5:19 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 5:19 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 5:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:19 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 5:19 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 5:20 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 5:20 p.m.
F821 undefined name 'calculateDistanceBetweenTwoDirs'	reviewbot	Nov. 8, 2018, 5:20 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 5:20 p.m.
E501 line too long (83 > 79 characters)	reviewbot	Nov. 8, 2018, 5:22 p.m.
F821 undefined name 'dist'	reviewbot	Nov. 8, 2018, 5:22 p.m.
F821 undefined name 'operator'	reviewbot	Nov. 8, 2018, 5:21 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:15 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:15 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 5:15 p.m.
E501 line too long (91 > 79 characters)	reviewbot	Nov. 8, 2018, 5:14 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:14 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 5:14 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 5:14 p.m.
W391 blank line at end of file	reviewbot	Nov. 8, 2018, 5:14 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 5:14 p.m.
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused	reviewbot	Nov. 8, 2018, 5:03 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:14 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 5:13 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 5:13 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:22 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 5:22 p.m.
F841 local variable 'filename1' is assigned to but never used	reviewbot	Nov. 8, 2018, 5:13 p.m.
F841 local variable 'filename2' is assigned to but never used	reviewbot	Nov. 8, 2018, 5:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 8, 2018, 5:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Nov. 8, 2018, 5:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 5:13 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 8:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 8:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 8:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 8:50 p.m.
E251 unexpected spaces around keyword / parameter equals	reviewbot	Nov. 11, 2018, 8:50 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:50 p.m.
E251 unexpected spaces around keyword / parameter equals	reviewbot	Nov. 11, 2018, 8:50 p.m.
E501 line too long (85 > 79 characters)	reviewbot	Nov. 11, 2018, 8:50 p.m.
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused	reviewbot	Nov. 8, 2018, 5:30 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 8:50 p.m.
E501 line too long (104 > 79 characters)	reviewbot	Nov. 11, 2018, 8:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 8:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 8:50 p.m.
E501 line too long (84 > 79 characters)	reviewbot	Nov. 11, 2018, 8:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:50 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:50 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:51 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Nov. 11, 2018, 8:51 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:51 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:51 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 8:51 p.m.
E501 line too long (90 > 79 characters)	reviewbot	Nov. 11, 2018, 8:51 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:51 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:54 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 8:55 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 8:55 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:03 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 8:56 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:57 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:57 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:03 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:57 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 8:58 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:59 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 8:59 p.m.
E501 line too long (90 > 79 characters)	reviewbot	Nov. 11, 2018, 8:59 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 8:59 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 8:59 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:59 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 9 p.m.
E501 line too long (82 > 79 characters)	reviewbot	Nov. 11, 2018, 9 p.m.
E271 multiple spaces after keyword	reviewbot	Nov. 11, 2018, 9:01 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:01 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 9:01 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:01 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 9:01 p.m.
E116 unexpected indentation (comment)	reviewbot	Nov. 11, 2018, 9:02 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 9:02 p.m.
W391 blank line at end of file	reviewbot	Nov. 11, 2018, 9:02 p.m.
E401 multiple imports on one line	reviewbot	Nov. 11, 2018, 9:03 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 11, 2018, 9:03 p.m.
E225 missing whitespace around operator	reviewbot	Nov. 11, 2018, 9:03 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:06 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 9:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:07 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:07 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 9:07 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 9:08 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Nov. 11, 2018, 9:08 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 9:08 p.m.
E501 line too long (84 > 79 characters)	reviewbot	Nov. 11, 2018, 9:09 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:09 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 9:14 p.m.
F841 local variable 'filename1' is assigned to but never used	reviewbot	Nov. 11, 2018, 9:10 p.m.
F841 local variable 'filename2' is assigned to but never used	reviewbot	Nov. 11, 2018, 9:10 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:10 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 11, 2018, 9:10 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:10 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:10 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 9:12 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:12 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Nov. 11, 2018, 9:12 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:13 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 9:13 p.m.
E301 expected 1 blank line, found 0	reviewbot	Nov. 11, 2018, 9:13 p.m.
E501 line too long (85 > 79 characters)	reviewbot	Nov. 11, 2018, 9:14 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 9:14 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 9:18 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 9:18 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 9:18 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 9:18 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:18 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 9:18 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 11, 2018, 9:18 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 11, 2018, 9:18 p.m.
E271 multiple spaces after keyword	reviewbot	Nov. 11, 2018, 9:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 9:18 p.m.
E303 too many blank lines (3)	reviewbot	Nov. 11, 2018, 9:18 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 9:18 p.m.
W503 line break before binary operator	reviewbot	Nov. 11, 2018, 9:18 p.m.
W292 no newline at end of file	reviewbot	Nov. 11, 2018, 9:18 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 11:36 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 11:37 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 11:37 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 11:37 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:38 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 11:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:39 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 11:39 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:39 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:39 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 11:39 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:39 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 11:39 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 11:39 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 11:39 a.m.
E231 missing whitespace after ':'	reviewbot	Nov. 19, 2018, 11:39 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 11:40 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 11:40 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 19, 2018, 11:42 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:42 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:42 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 11:43 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 11:43 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:43 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:43 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 11:43 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:43 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:43 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 11:43 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:43 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 11:43 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 11:43 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 11:43 a.m.
E231 missing whitespace after ':'	reviewbot	Nov. 19, 2018, 11:43 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 11:43 a.m.
You should add a trailing comma over here	bolariinwa	Nov. 19, 2018, 11:46 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 11:43 a.m.
F401 'pdb' imported but unused	reviewbot	Nov. 19, 2018, 11:43 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 19, 2018, 11:43 a.m.
You're creating a "Model" here, but that's not really what you want. Django models correspond to database tables, and you're …	david	Nov. 19, 2018, 11:53 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:44 a.m.
E303 too many blank lines (2)	reviewbot	Nov. 19, 2018, 11:44 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:44 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 11:44 a.m.
You have some trailing whitespace over here	bolariinwa	Nov. 19, 2018, 11:46 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 11:44 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:44 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 11:44 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 11:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 11:45 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 11:45 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:45 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 11:45 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 11:45 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 11:45 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 19, 2018, 11:55 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 11:57 a.m.
F401 'suggest_reviewers_extension.SuggestReviewersResource.suggestReviewersResource' imported but unused	reviewbot	Nov. 19, 2018, 11:57 a.m.
F401 'djblets.webapi.resources.unregister_resource_for_model' imported but unused	reviewbot	Nov. 19, 2018, 11:57 a.m.
F401 'djblets.webapi.resources.register_resource_for_model' imported but unused	reviewbot	Nov. 19, 2018, 11:57 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 11:57 a.m.
E231 missing whitespace after ','	reviewbot	Nov. 19, 2018, 11:58 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 2:46 p.m.
E231 missing whitespace after ','	reviewbot	Nov. 19, 2018, 2:46 p.m.
Is it worth using a Counter here instead of a Dict? I don't know if this will actually boost performance …	gojeffcho	Dec. 6, 2018, 7:32 p.m.
I know that this is WIP so a lot of these comments could be self-notes so you can keep track …	gojeffcho	Nov. 29, 2018, 3:57 p.m.
Ditto some of your variable naming, some of them don't conform to standards and others are unclear/unspecific like download_dir being …	gojeffcho	Nov. 29, 2018, 3:58 p.m.
This seems to be hardcoded into a lot of these files - could it be centralized somewhere and retrieved instead …	gojeffcho	Dec. 6, 2018, 7:32 p.m.
I believe this docstring should end with a period, and the linebreaks look a bit off to me in the …	gojeffcho	Dec. 6, 2018, 7:32 p.m.
Is it worth checking that the files exist before this?	gojeffcho	Nov. 29, 2018, 7:11 p.m.
It seems like Counter could work well here as well.	gojeffcho	Dec. 6, 2018, 7:32 p.m.
Does an if not review_request.get_draft() fail here sometimes even when it's not None?	gojeffcho	Dec. 6, 2018, 7:32 p.m.
E902 IndentationError: unindent does not match any outer indentation level	reviewbot	Dec. 6, 2018, 7:29 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Dec. 6, 2018, 7:19 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 7:19 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 6, 2018, 7:19 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 7:20 p.m.
E999 IndentationError: unindent does not match any outer indentation level	reviewbot	Dec. 6, 2018, 7:20 p.m.
F401 'djblets.util.fields.JSONField' imported but unused	reviewbot	Dec. 6, 2018, 7:21 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 6, 2018, 7:24 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 7:24 p.m.
W391 blank line at end of file	reviewbot	Dec. 6, 2018, 7:24 p.m.
Col: 9 'self' is defined but never used.	reviewbot	Dec. 6, 2018, 7:24 p.m.
E231 missing whitespace after ','	reviewbot	Dec. 6, 2018, 7:28 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 6, 2018, 7:29 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 6, 2018, 7:29 p.m.
E304 blank lines found after function decorator	reviewbot	Dec. 6, 2018, 7:29 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 7:30 p.m.
E902 TokenError: EOF in multi-line statement	reviewbot	Dec. 6, 2018, 7:30 p.m.
E123 closing bracket does not match indentation of opening bracket's line	reviewbot	Dec. 6, 2018, 7:31 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Dec. 6, 2018, 7:31 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Dec. 6, 2018, 7:31 p.m.
E123 closing bracket does not match indentation of opening bracket's line	reviewbot	Dec. 6, 2018, 7:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:32 p.m.
E113 unexpected indentation	reviewbot	Dec. 6, 2018, 7:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 7:32 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 7:40 p.m.
E304 blank lines found after function decorator	reviewbot	Dec. 6, 2018, 7:40 p.m.
W292 no newline at end of file	reviewbot	Dec. 6, 2018, 7:40 p.m.
F401 'reviewboard.extensions.base.JSExtension' imported but unused	reviewbot	Dec. 6, 2018, 7:40 p.m.
F821 undefined name 'SuggestReviewersJSExtension'	reviewbot	Dec. 6, 2018, 7:40 p.m.
F821 undefined name 'SuggestReviewersJSExtension'	reviewbot	Dec. 6, 2018, 7:41 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Dec. 6, 2018, 7:41 p.m.
F821 undefined name 'suggestReviewersResource'	reviewbot	Dec. 6, 2018, 7:42 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 7:51 p.m.
F821 undefined name 'suggest_reviewers_resource'	reviewbot	Dec. 6, 2018, 7:44 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 7:51 p.m.
Trailing whitespace here	bolariinwa	Dec. 8, 2018, 6:52 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 6:54 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 6:52 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 6:54 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 6:47 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 6:47 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 6:47 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 8, 2018, 6:47 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 6:54 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 6:54 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 6:54 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 8, 2018, 6:54 p.m.
This file is not a model. I'm confused as to its location (and why an extension would have RB models?)	brennie	Dec. 13, 2018, 7:17 a.m.
This is a class, which isn't intended to contain arbitrary code. In Java, everything must be contained in a class, …	brennie	Dec. 13, 2018, 7:17 a.m.
This file is not a model. I'm confused as to its location (and why an extension would have RB models?)	brennie	Dec. 13, 2018, 7:17 a.m.
This file is not a model. I'm confused as to its location (and why an extension would have RB models?)	brennie	Dec. 13, 2018, 7:17 a.m.
Docstrings should be of the format: """Single line summary. Multi-line description. """	brennie	Dec. 13, 2018, 7:17 a.m.
F401 'djblets.webapi.decorators.webapi_request_fields' imported but unused	reviewbot	Dec. 11, 2018, 11:33 p.m.
F401 'reviewboard.webapi.decorators.webapi_login_required' imported but unused	reviewbot	Dec. 11, 2018, 11:33 p.m.
F401 'reviewboard.webapi.decorators.webapi_check_local_site' imported but unused	reviewbot	Dec. 11, 2018, 11:33 p.m.
F401 'reviewboard.webapi.resources.resources' imported but unused	reviewbot	Dec. 11, 2018, 11:33 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 11, 2018, 11:34 p.m.
E305 expected 2 blank lines after class or function definition, found 1	reviewbot	Dec. 11, 2018, 11:34 p.m.
F401 'reviewboard.extensions.base.JSExtension' imported but unused	reviewbot	Dec. 11, 2018, 11:34 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 11, 2018, 11:46 p.m.
E203 whitespace before ':'	reviewbot	Dec. 11, 2018, 11:46 p.m.
W291 trailing whitespace	reviewbot	Dec. 11, 2018, 11:47 p.m.
W292 no newline at end of file	reviewbot	Dec. 11, 2018, 11:47 p.m.
Col: 53 Expected '!==' and instead saw '!='.	reviewbot	Dec. 11, 2018, 11:47 p.m.
W291 trailing whitespace	reviewbot	Dec. 12, 2018, 1:56 p.m.
E225 missing whitespace around operator	reviewbot	Dec. 13, 2018, 7:04 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 13, 2018, 7:04 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 7:04 a.m.
F401 'pdb' imported but unused	reviewbot	Dec. 13, 2018, 7:05 a.m.
E231 missing whitespace after ':'	reviewbot	Dec. 13, 2018, 7:05 a.m.
E231 missing whitespace after ':'	reviewbot	Dec. 13, 2018, 7:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:05 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 7:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 7:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 7:06 a.m.
W291 trailing whitespace	reviewbot	Dec. 13, 2018, 7:06 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 7:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:07 a.m.
E201 whitespace after '('	reviewbot	Dec. 13, 2018, 7:07 a.m.
E225 missing whitespace around operator	reviewbot	Dec. 13, 2018, 7:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:07 a.m.
W293 blank line contains whitespace	reviewbot	Dec. 13, 2018, 7:16 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 7:08 a.m.
W293 blank line contains whitespace	reviewbot	Dec. 13, 2018, 7:16 a.m.
E231 missing whitespace after ','	reviewbot	Dec. 13, 2018, 7:10 a.m.
E305 expected 2 blank lines after class or function definition, found 1	reviewbot	Dec. 13, 2018, 7:10 a.m.
W292 no newline at end of file	reviewbot	Dec. 13, 2018, 7:10 a.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
E401 multiple imports on one line
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W601 .has_key() is deprecated, use 'in'
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
E501 line too long (88 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
E501 line too long (94 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
Show all issues
```
W391 blank line at end of file
```

Commit:

-	646a68b3cca782a80d2dc13a941ac8921cd9fa3e
+	fd379e16b3b56e5dc5088dd192f110aba3eee90b

Diff:

Revision 2 (+214)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	4 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 43 failures.
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 2)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 2)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
E501 line too long (102 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
E501 line too long (117 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
Show all issues
```
W291 trailing whitespace
```

Commit:

-	fd379e16b3b56e5dc5088dd192f110aba3eee90b
+	a0396bb3287bfd442926fa5a6b56fe947f21b669

Diff:

Revision 3 (+258)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 46 failures.
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 3)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 3)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
F401 'reviewboard.urls.diffviewer_url_names' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
E302 expected 2 blank lines, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
E501 line too long (95 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
Show all issues
```
W291 trailing whitespace
```

Commit:

-	a0396bb3287bfd442926fa5a6b56fe947f21b669
+	a8b29915c1a37b5742a0520bdbca79959e5d413e

Diff:

Revision 4 (+249)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

reviewboard/reviews/models/countReviewers.py (Diff revision 4)
Show all issues
```
E501 line too long (81 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 4)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 4)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
F401 'reviewboard.urls.diffviewer_url_names' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
E302 expected 2 blank lines, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
E501 line too long (95 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
E501 line too long (92 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
F401 'djblets.util.fields.JSONField' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
F401 'reviewboard.reviews.models.ReviewRequest' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
F401 'reviewboard.reviews.models.Review' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
Show all issues
```
W292 no newline at end of file
```

Commit:

-	a8b29915c1a37b5742a0520bdbca79959e5d413e
+	c807e0fd31ae4632a6139f4c6ba1108f6d41f5a4

Diff:

Revision 5 (+255)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

reviewboard/reviews/models/countReviewers.py (Diff revision 5)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 5)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E225 missing whitespace around operator
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 5)
Show all issues
```
F821 undefined name 'models'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 5)
Show all issues
```
W292 no newline at end of file
```

Commit:

-	c807e0fd31ae4632a6139f4c6ba1108f6d41f5a4
+	4957f09647642ea073b3b6f26adc625a88f7f877

Diff:

Revision 6 (+259)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/setup.py (Diff revision 6)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
Show all issues
```
E127 continuation line over-indented for visual indent
```

Commit:

-	4957f09647642ea073b3b6f26adc625a88f7f877
+	12baebd145259fdec3520f93b9bb58b61b260d5a

Diff:

Revision 7 (+261)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/setup.py (Diff revision 7)
Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 7)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 7)
Show all issues
```
E127 continuation line over-indented for visual indent
```

Commit:

-	12baebd145259fdec3520f93b9bb58b61b260d5a
+	f2dcb03cc987a49e48293eeba678b05e061d2430

Diff:

Revision 8 (+258)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/setup.py (Diff revision 8)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
Show all issues
```
E703 statement ends with a semicolon
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 8)
Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```

Description:

~		When a user makes a review request, if they enable this extension then they can
~		get a suggestion about a reviewer that would be the best to review the submitted
~		code, given the files that this user would like to review.
	~	Summary of what this extension would do: When a user makes a review request,
	~	if they enable this extension then they can get a suggestion about a reviewer
	~	that would be the best to review the submitted code, given the files that
	+	this user would like to review.
	+
	+	What I have done:
	+	Created an extension folder called `suggested_reviewers_extension` and added an
	+	`extension.py` that defines the basis of the extension. The file also creates
	+	a `JSExtension` subclass that stores the right reviewer(s) in a variable that was
	+	calculated from the `models.py` file so that the `template.HTML` can be dynamic
	+	depending on who is meant to review the request.
	+
	+	I also created a `template.HTML` class that displays the suggested reviewer or on
	+	default, displays that there is no one to suggest. Also, the template gets the DOM
	+	element of the review-request-details and attaches the suggested_review div to that
	+	element.
	+
	+	I'm working on the `models.py` file now to correctly return the right reviewer based
	+	on the right weights and right variables.

Commit:

-	f2dcb03cc987a49e48293eeba678b05e061d2430
+	f409b45081803b7825903af0930da5f9e261de50

Diff:

Revision 9 (+357)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 36 failures.
```
suggest_reviewers_extension/setup.py (Diff revision 9)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
F821 undefined name 'calculateDistanceBetweenTwoDirs'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (83 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
F821 undefined name 'dist'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
F821 undefined name 'operator'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E114 indentation is not a multiple of four (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E116 unexpected indentation (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
F841 local variable 'review_request' is assigned to but never used
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
E501 line too long (81 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
Show all issues
```
W293 blank line contains whitespace
```

Commit:

-	f409b45081803b7825903af0930da5f9e261de50
+	b478a7e96d6671a30339967df067ed346cbd962d

Diff:

Revision 10 (+440)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/retrieveInformation.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	8 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 58 failures.
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
E303 too many blank lines (2)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
E501 line too long (91 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 10)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
F821 undefined name 'calculateDistanceBetweenTwoDirs'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
E501 line too long (83 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
F821 undefined name 'dist'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
Show all issues
```
F821 undefined name 'operator'
```

Summary:

-	[WIP]: Create a Suggestive reviewers Extension
+	[WIP]: Create an Extension that suggests reviewers

Commit:

-	b478a7e96d6671a30339967df067ed346cbd962d
+	ad6cf8b15d96ba4b7cfec679342716996b883148

Diff:

Revision 11 (+475)

Show changes

	data.csv
	reviewboard/reviews/models/countReviewers.py
	reviewboard/reviews/models/retrieveInformation.py
	reviewboard/reviews/models/reviewers.py
	suggest_reviewers_extension/MANIFEST.in
	suggest_reviewers_extension/README.rst
	suggest_reviewers_extension/setup.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	8 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 89 failures.
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
E303 too many blank lines (2)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
E501 line too long (91 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 11)
Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
Show all issues
```
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)