Summary

Create an Extension that suggests reviewers

Review Request #10248 — Created Oct. 19, 2018 and discarded March 8, 2019, 4: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, 9:24 a.m.
E401 multiple imports on one line	reviewbot	Oct. 24, 2018, 8:01 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:02 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:02 p.m.
W601 .has_key() is deprecated, use 'in'	reviewbot	Oct. 24, 2018, 8:02 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:03 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:03 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:03 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:08 p.m.
E501 line too long (88 > 79 characters)	reviewbot	Oct. 24, 2018, 8:08 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 24, 2018, 8:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:08 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:08 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 8:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:08 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 8:08 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:08 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:08 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:08 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 8:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 8:09 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:10 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 8:09 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:09 p.m.
E501 line too long (102 > 79 characters)	reviewbot	Oct. 24, 2018, 8:09 p.m.
E501 line too long (117 > 79 characters)	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 8:09 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 8:09 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:24 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:24 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
E265 block comment should start with '# '	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:24 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 8:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:24 p.m.
F401 'django.db.models' imported but unused	reviewbot	Oct. 24, 2018, 8:24 p.m.
E402 module level import not at top of file	reviewbot	Oct. 24, 2018, 8:24 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:24 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:24 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:24 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:37 p.m.
F401 'reviewboard.urls.diffviewer_url_names' imported but unused	reviewbot	Oct. 24, 2018, 8:37 p.m.
E303 too many blank lines (3)	reviewbot	Oct. 24, 2018, 8:37 p.m.
E302 expected 2 blank lines, found 0	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:37 p.m.
F401 'reviewboard.urls.diffviewer_url_names' imported but unused	reviewbot	Oct. 24, 2018, 8:37 p.m.
E303 too many blank lines (3)	reviewbot	Oct. 24, 2018, 8:37 p.m.
E302 expected 2 blank lines, found 0	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:37 p.m.
E501 line too long (92 > 79 characters)	reviewbot	Oct. 24, 2018, 8:37 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 8:37 p.m.
F401 'django.contrib.auth.models.User' imported but unused	reviewbot	Oct. 24, 2018, 8:37 p.m.
F401 'djblets.util.fields.JSONField' imported but unused	reviewbot	Oct. 24, 2018, 8:38 p.m.
F401 'reviewboard.reviews.models.ReviewRequest' imported but unused	reviewbot	Oct. 24, 2018, 8:38 p.m.
F401 'reviewboard.reviews.models.Review' imported but unused	reviewbot	Oct. 24, 2018, 8:38 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:38 p.m.
W293 blank line contains whitespace	reviewbot	Oct. 24, 2018, 8:38 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 8:38 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Oct. 24, 2018, 8:38 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 24, 2018, 8:38 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 8:38 p.m.
W391 blank line at end of file	reviewbot	Oct. 24, 2018, 8:46 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 8:40 p.m.
E301 expected 1 blank line, found 0	reviewbot	Oct. 24, 2018, 8:46 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:46 p.m.
E225 missing whitespace around operator	reviewbot	Oct. 24, 2018, 8:46 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 8:46 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:47 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 8:47 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 24, 2018, 8:47 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Oct. 24, 2018, 8:47 p.m.
F821 undefined name 'models'	reviewbot	Oct. 24, 2018, 8:47 p.m.
W292 no newline at end of file	reviewbot	Oct. 24, 2018, 8:47 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 24, 2018, 8:50 p.m.
W291 trailing whitespace	reviewbot	Oct. 24, 2018, 8:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 24, 2018, 8:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 24, 2018, 8:50 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 8, 2018, 4:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:15 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 4:15 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:15 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 4:15 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:15 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:15 p.m.
E703 statement ends with a semicolon	reviewbot	Nov. 8, 2018, 4:15 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:16 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:16 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 4:16 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 4:16 p.m.
F821 undefined name 'calculateDistanceBetweenTwoDirs'	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
E501 line too long (83 > 79 characters)	reviewbot	Nov. 8, 2018, 4:16 p.m.
F821 undefined name 'dist'	reviewbot	Nov. 8, 2018, 4:16 p.m.
F821 undefined name 'operator'	reviewbot	Nov. 8, 2018, 4:16 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:17 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 8, 2018, 4:17 p.m.
E116 unexpected indentation (comment)	reviewbot	Nov. 8, 2018, 4:17 p.m.
F841 local variable 'review_request' is assigned to but never used	reviewbot	Nov. 8, 2018, 4:17 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Nov. 8, 2018, 4:17 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:17 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:18 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 4:18 p.m.
E501 line too long (91 > 79 characters)	reviewbot	Nov. 8, 2018, 4:18 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 4:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:18 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 4:18 p.m.
W391 blank line at end of file	reviewbot	Nov. 8, 2018, 4:18 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 4:18 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:18 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 4:18 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:18 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:18 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:19 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:19 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 4:19 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 4:19 p.m.
E501 line too long (93 > 79 characters)	reviewbot	Nov. 8, 2018, 4:19 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:19 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 4:19 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 4:20 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 8, 2018, 4:20 p.m.
F821 undefined name 'calculateDistanceBetweenTwoDirs'	reviewbot	Nov. 8, 2018, 4:20 p.m.
E501 line too long (89 > 79 characters)	reviewbot	Nov. 8, 2018, 4:20 p.m.
E501 line too long (83 > 79 characters)	reviewbot	Nov. 8, 2018, 4:22 p.m.
F821 undefined name 'dist'	reviewbot	Nov. 8, 2018, 4:22 p.m.
F821 undefined name 'operator'	reviewbot	Nov. 8, 2018, 4:21 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:15 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:15 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 8, 2018, 4:15 p.m.
E501 line too long (91 > 79 characters)	reviewbot	Nov. 8, 2018, 4:14 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:14 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 4:14 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 4:14 p.m.
W391 blank line at end of file	reviewbot	Nov. 8, 2018, 4:14 p.m.
E501 line too long (108 > 79 characters)	reviewbot	Nov. 8, 2018, 4:14 p.m.
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused	reviewbot	Nov. 8, 2018, 4:03 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:14 p.m.
E501 line too long (110 > 79 characters)	reviewbot	Nov. 8, 2018, 4:13 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 8, 2018, 4:13 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:22 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 8, 2018, 4:22 p.m.
F841 local variable 'filename1' is assigned to but never used	reviewbot	Nov. 8, 2018, 4:13 p.m.
F841 local variable 'filename2' is assigned to but never used	reviewbot	Nov. 8, 2018, 4:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 8, 2018, 4:13 p.m.
W291 trailing whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Nov. 8, 2018, 4:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 8, 2018, 4:13 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 7:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 7:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 7:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E111 indentation is not a multiple of four	reviewbot	Nov. 11, 2018, 7:50 p.m.
E251 unexpected spaces around keyword / parameter equals	reviewbot	Nov. 11, 2018, 7:50 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 7:50 p.m.
E251 unexpected spaces around keyword / parameter equals	reviewbot	Nov. 11, 2018, 7:50 p.m.
E501 line too long (85 > 79 characters)	reviewbot	Nov. 11, 2018, 7:50 p.m.
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused	reviewbot	Nov. 8, 2018, 4:30 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 7:50 p.m.
E501 line too long (104 > 79 characters)	reviewbot	Nov. 11, 2018, 7:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 7:50 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 7:50 p.m.
E501 line too long (84 > 79 characters)	reviewbot	Nov. 11, 2018, 7:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:50 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 7:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:50 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 7:50 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:51 p.m.
E501 line too long (81 > 79 characters)	reviewbot	Nov. 11, 2018, 7:51 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:51 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 7:51 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 7:51 p.m.
E501 line too long (90 > 79 characters)	reviewbot	Nov. 11, 2018, 7:51 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 7:51 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:54 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 7:55 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 7:55 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:03 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 7:56 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:57 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:57 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:03 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 7:57 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 7:58 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 7:59 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 7:59 p.m.
E501 line too long (90 > 79 characters)	reviewbot	Nov. 11, 2018, 7:59 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 7:59 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 7:59 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 7:59 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 8 p.m.
E501 line too long (82 > 79 characters)	reviewbot	Nov. 11, 2018, 8 p.m.
E271 multiple spaces after keyword	reviewbot	Nov. 11, 2018, 8:01 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:01 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 8:01 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:01 p.m.
E114 indentation is not a multiple of four (comment)	reviewbot	Nov. 11, 2018, 8:01 p.m.
E116 unexpected indentation (comment)	reviewbot	Nov. 11, 2018, 8:02 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 8:02 p.m.
W391 blank line at end of file	reviewbot	Nov. 11, 2018, 8:02 p.m.
E401 multiple imports on one line	reviewbot	Nov. 11, 2018, 8:03 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 11, 2018, 8:03 p.m.
E225 missing whitespace around operator	reviewbot	Nov. 11, 2018, 8:03 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:06 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 8:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:07 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:07 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:07 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:07 p.m.
E113 unexpected indentation	reviewbot	Nov. 11, 2018, 8:08 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Nov. 11, 2018, 8:08 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 8:08 p.m.
E501 line too long (84 > 79 characters)	reviewbot	Nov. 11, 2018, 8:09 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:09 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 8:14 p.m.
F841 local variable 'filename1' is assigned to but never used	reviewbot	Nov. 11, 2018, 8:10 p.m.
F841 local variable 'filename2' is assigned to but never used	reviewbot	Nov. 11, 2018, 8:10 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:10 p.m.
E501 line too long (87 > 79 characters)	reviewbot	Nov. 11, 2018, 8:10 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:10 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:10 p.m.
E501 line too long (99 > 79 characters)	reviewbot	Nov. 11, 2018, 8:12 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:12 p.m.
E501 line too long (94 > 79 characters)	reviewbot	Nov. 11, 2018, 8:12 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:13 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:13 p.m.
E303 too many blank lines (2)	reviewbot	Nov. 11, 2018, 8:13 p.m.
E301 expected 1 blank line, found 0	reviewbot	Nov. 11, 2018, 8:13 p.m.
E501 line too long (85 > 79 characters)	reviewbot	Nov. 11, 2018, 8:14 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 11, 2018, 8:14 p.m.
W291 trailing whitespace	reviewbot	Nov. 11, 2018, 8:18 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 11, 2018, 8:18 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 11, 2018, 8:18 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:18 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:18 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 11, 2018, 8:18 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 11, 2018, 8:18 p.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 11, 2018, 8:18 p.m.
E271 multiple spaces after keyword	reviewbot	Nov. 11, 2018, 8:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:18 p.m.
W293 blank line contains whitespace	reviewbot	Nov. 11, 2018, 8:18 p.m.
E303 too many blank lines (3)	reviewbot	Nov. 11, 2018, 8:18 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 11, 2018, 8:18 p.m.
W503 line break before binary operator	reviewbot	Nov. 11, 2018, 8:18 p.m.
W292 no newline at end of file	reviewbot	Nov. 11, 2018, 8:18 p.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 10:36 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 10:37 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 10:37 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 10:37 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:38 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 10:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:39 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 10:39 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:39 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:39 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 10:39 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:39 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 10:39 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 10:39 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 10:39 a.m.
E231 missing whitespace after ':'	reviewbot	Nov. 19, 2018, 10:39 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 10:40 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 10:40 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 19, 2018, 10:42 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:42 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:42 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 10:43 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 10:43 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:43 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:43 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 10:43 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:43 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:43 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 10:43 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:43 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 10:43 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 10:43 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 10:43 a.m.
E231 missing whitespace after ':'	reviewbot	Nov. 19, 2018, 10:43 a.m.
E131 continuation line unaligned for hanging indent	reviewbot	Nov. 19, 2018, 10:43 a.m.
You should add a trailing comma over here	bolariinwa	Nov. 19, 2018, 10:46 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 10:43 a.m.
F401 'pdb' imported but unused	reviewbot	Nov. 19, 2018, 10:43 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Nov. 19, 2018, 10: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, 10:53 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:44 a.m.
E303 too many blank lines (2)	reviewbot	Nov. 19, 2018, 10:44 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:44 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 10:44 a.m.
You have some trailing whitespace over here	bolariinwa	Nov. 19, 2018, 10:46 a.m.
E999 SyntaxError: invalid syntax	reviewbot	Nov. 19, 2018, 10:44 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:44 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Nov. 19, 2018, 10:44 a.m.
E271 multiple spaces after keyword	reviewbot	Nov. 19, 2018, 10:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:44 a.m.
W293 blank line contains whitespace	reviewbot	Nov. 19, 2018, 10:45 a.m.
E303 too many blank lines (3)	reviewbot	Nov. 19, 2018, 10:45 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:45 a.m.
W503 line break before binary operator	reviewbot	Nov. 19, 2018, 10:45 a.m.
W292 no newline at end of file	reviewbot	Nov. 19, 2018, 10:45 a.m.
F401 'django.http.HttpResponse' imported but unused	reviewbot	Nov. 19, 2018, 10:45 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Nov. 19, 2018, 10:55 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Nov. 19, 2018, 10:57 a.m.
F401 'suggest_reviewers_extension.SuggestReviewersResource.suggestReviewersResource' imported but unused	reviewbot	Nov. 19, 2018, 10:57 a.m.
F401 'djblets.webapi.resources.unregister_resource_for_model' imported but unused	reviewbot	Nov. 19, 2018, 10:57 a.m.
F401 'djblets.webapi.resources.register_resource_for_model' imported but unused	reviewbot	Nov. 19, 2018, 10:57 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Nov. 19, 2018, 10:57 a.m.
E231 missing whitespace after ','	reviewbot	Nov. 19, 2018, 10:58 a.m.
W291 trailing whitespace	reviewbot	Nov. 19, 2018, 1:46 p.m.
E231 missing whitespace after ','	reviewbot	Nov. 19, 2018, 1: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, 6: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, 2: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, 2: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, 6: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, 6:32 p.m.
Is it worth checking that the files exist before this?	gojeffcho	Nov. 29, 2018, 6:11 p.m.
It seems like Counter could work well here as well.	gojeffcho	Dec. 6, 2018, 6:32 p.m.
Does an if not review_request.get_draft() fail here sometimes even when it's not None?	gojeffcho	Dec. 6, 2018, 6:32 p.m.
E902 IndentationError: unindent does not match any outer indentation level	reviewbot	Dec. 6, 2018, 6:29 p.m.
E127 continuation line over-indented for visual indent	reviewbot	Dec. 6, 2018, 6:19 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 6:19 p.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 6, 2018, 6:19 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 6:20 p.m.
E999 IndentationError: unindent does not match any outer indentation level	reviewbot	Dec. 6, 2018, 6:20 p.m.
F401 'djblets.util.fields.JSONField' imported but unused	reviewbot	Dec. 6, 2018, 6:21 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 6, 2018, 6:24 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 6, 2018, 6:24 p.m.
W391 blank line at end of file	reviewbot	Dec. 6, 2018, 6:24 p.m.
Col: 9 'self' is defined but never used.	reviewbot	Dec. 6, 2018, 6:24 p.m.
E231 missing whitespace after ','	reviewbot	Dec. 6, 2018, 6:28 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 6, 2018, 6:29 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 6, 2018, 6:29 p.m.
E304 blank lines found after function decorator	reviewbot	Dec. 6, 2018, 6:29 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 6:30 p.m.
E902 TokenError: EOF in multi-line statement	reviewbot	Dec. 6, 2018, 6:30 p.m.
E123 closing bracket does not match indentation of opening bracket's line	reviewbot	Dec. 6, 2018, 6:31 p.m.
E999 SyntaxError: invalid syntax	reviewbot	Dec. 6, 2018, 6:31 p.m.
E131 continuation line unaligned for hanging indent	reviewbot	Dec. 6, 2018, 6:31 p.m.
E123 closing bracket does not match indentation of opening bracket's line	reviewbot	Dec. 6, 2018, 6:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:31 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:32 p.m.
E113 unexpected indentation	reviewbot	Dec. 6, 2018, 6:32 p.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 6, 2018, 6:32 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 6:40 p.m.
E304 blank lines found after function decorator	reviewbot	Dec. 6, 2018, 6:40 p.m.
W292 no newline at end of file	reviewbot	Dec. 6, 2018, 6:40 p.m.
F401 'reviewboard.extensions.base.JSExtension' imported but unused	reviewbot	Dec. 6, 2018, 6:40 p.m.
F821 undefined name 'SuggestReviewersJSExtension'	reviewbot	Dec. 6, 2018, 6:40 p.m.
F821 undefined name 'SuggestReviewersJSExtension'	reviewbot	Dec. 6, 2018, 6:41 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Dec. 6, 2018, 6:41 p.m.
F821 undefined name 'suggestReviewersResource'	reviewbot	Dec. 6, 2018, 6:42 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 6:51 p.m.
F821 undefined name 'suggest_reviewers_resource'	reviewbot	Dec. 6, 2018, 6:44 p.m.
E999 IndentationError: unexpected unindent	reviewbot	Dec. 6, 2018, 6:51 p.m.
Trailing whitespace here	bolariinwa	Dec. 8, 2018, 5:52 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 5:54 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 5:52 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 5:54 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 5:47 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 5:47 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 5:47 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 8, 2018, 5:47 p.m.
W291 trailing whitespace	reviewbot	Dec. 8, 2018, 5:54 p.m.
E301 expected 1 blank line, found 0	reviewbot	Dec. 8, 2018, 5:54 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 8, 2018, 5:54 p.m.
W293 blank line contains whitespace	reviewbot	Dec. 8, 2018, 5: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, 6: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, 6: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, 6: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, 6:17 a.m.
Docstrings should be of the format: """Single line summary. Multi-line description. """	brennie	Dec. 13, 2018, 6:17 a.m.
F401 'djblets.webapi.decorators.webapi_request_fields' imported but unused	reviewbot	Dec. 11, 2018, 10:33 p.m.
F401 'reviewboard.webapi.decorators.webapi_login_required' imported but unused	reviewbot	Dec. 11, 2018, 10:33 p.m.
F401 'reviewboard.webapi.decorators.webapi_check_local_site' imported but unused	reviewbot	Dec. 11, 2018, 10:33 p.m.
F401 'reviewboard.webapi.resources.resources' imported but unused	reviewbot	Dec. 11, 2018, 10:33 p.m.
E231 missing whitespace after ':'	reviewbot	Dec. 11, 2018, 10:34 p.m.
E305 expected 2 blank lines after class or function definition, found 1	reviewbot	Dec. 11, 2018, 10:34 p.m.
F401 'reviewboard.extensions.base.JSExtension' imported but unused	reviewbot	Dec. 11, 2018, 10:34 p.m.
E501 line too long (80 > 79 characters)	reviewbot	Dec. 11, 2018, 10:46 p.m.
E203 whitespace before ':'	reviewbot	Dec. 11, 2018, 10:46 p.m.
W291 trailing whitespace	reviewbot	Dec. 11, 2018, 10:47 p.m.
W292 no newline at end of file	reviewbot	Dec. 11, 2018, 10:47 p.m.
Col: 53 Expected '!==' and instead saw '!='.	reviewbot	Dec. 11, 2018, 10:47 p.m.
W291 trailing whitespace	reviewbot	Dec. 12, 2018, 12:56 p.m.
E225 missing whitespace around operator	reviewbot	Dec. 13, 2018, 6:04 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Dec. 13, 2018, 6:04 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 6:04 a.m.
F401 'pdb' imported but unused	reviewbot	Dec. 13, 2018, 6:05 a.m.
E231 missing whitespace after ':'	reviewbot	Dec. 13, 2018, 6:05 a.m.
E231 missing whitespace after ':'	reviewbot	Dec. 13, 2018, 6:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:05 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:05 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 6:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 6:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 6:06 a.m.
W291 trailing whitespace	reviewbot	Dec. 13, 2018, 6:06 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:06 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Dec. 13, 2018, 6:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:07 a.m.
E201 whitespace after '('	reviewbot	Dec. 13, 2018, 6:07 a.m.
E225 missing whitespace around operator	reviewbot	Dec. 13, 2018, 6:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:07 a.m.
W293 blank line contains whitespace	reviewbot	Dec. 13, 2018, 6:16 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Dec. 13, 2018, 6:08 a.m.
W293 blank line contains whitespace	reviewbot	Dec. 13, 2018, 6:16 a.m.
E231 missing whitespace after ','	reviewbot	Dec. 13, 2018, 6:10 a.m.
E305 expected 2 blank lines after class or function definition, found 1	reviewbot	Dec. 13, 2018, 6:10 a.m.
W292 no newline at end of file	reviewbot	Dec. 13, 2018, 6: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

Create an Extension that suggests reviewers

Information

Reviewers

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Description:

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Summary:

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commit:

Diff: