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)
The issue has been resolved. Show all issues
```
E401 multiple imports on one line
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W601 .has_key() is deprecated, use 'in'
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (88 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (94 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 1)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E501 line too long (102 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E501 line too long (117 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 2)
The issue has been resolved. 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)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E265 block comment should start with '# '
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
W391 blank line at end of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
F401 'django.db.models' imported but unused
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been dropped. Show all issues
```
E402 module level import not at top of file
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/reviewers.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
F401 'reviewboard.urls.diffviewer_url_names' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (95 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 3)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
E501 line too long (81 > 79 characters)
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/countReviewers.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/__init__.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'reviewboard.urls.diffviewer_url_names' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E501 line too long (95 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E501 line too long (92 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'django.contrib.auth.models.User' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'djblets.util.fields.JSONField' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'reviewboard.reviews.models.ReviewRequest' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'reviewboard.reviews.models.Review' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 4)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
The issue has been resolved. Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 4)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 5)
The issue has been dropped. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E225 missing whitespace around operator
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 5)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 5)
The issue has been resolved. Show all issues
```
F821 undefined name 'models'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 5)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 6)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 7)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 7)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 8)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
The issue has been resolved. Show all issues
```
E703 statement ends with a semicolon
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 8)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 8)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
F821 undefined name 'calculateDistanceBetweenTwoDirs'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E501 line too long (83 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
F821 undefined name 'dist'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
F821 undefined name 'operator'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. Show all issues
```
E116 unexpected indentation (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been dropped. 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)
The issue has been resolved. Show all issues
```
E501 line too long (81 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 9)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (91 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (93 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
F821 undefined name 'calculateDistanceBetweenTwoDirs'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (89 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
E501 line too long (83 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. Show all issues
```
F821 undefined name 'dist'
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 10)
The issue has been resolved. 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)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (91 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/setup.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (108 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
The issue has been dropped. Show all issues
```
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (110 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
F841 local variable 'filename1' is assigned to but never used
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
F841 local variable 'filename2' is assigned to but never used
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (99 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
E501 line too long (94 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 11)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```

Commit:

ad6cf8b15d96ba4b7cfec679342716996b883148

c080c8705c2967ed17f564e4f4b76fe369eebbd1

Diff:

Revision 12 (+479)

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 78 failures.
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E111 indentation is not a multiple of four
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E111 indentation is not a multiple of four
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E111 indentation is not a multiple of four
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
reviewboard/reviews/models/retrieveInformation.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E111 indentation is not a multiple of four
```
suggest_reviewers_extension/setup.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E251 unexpected spaces around keyword / parameter equals
```
suggest_reviewers_extension/setup.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/setup.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E251 unexpected spaces around keyword / parameter equals
```
suggest_reviewers_extension/setup.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (85 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been dropped. Show all issues
```
F401 'suggest_reviewers_extension.models.SuggestReviewers' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (104 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (84 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (81 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E113 unexpected indentation
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
E501 line too long (90 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 12)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```

The issue has been resolved. Show all issues

Hey Praise! I looked at your code, and even after reading the Review Board documentation for extensions I still don't really know a lot about extensions. So I don't really have anything specific to comment on, but I did want to ask a question.

Have you put thought into when the extension is going to run? Like, if it runs when you click on a Review Request, or something else? That could be a good place to start. Then you'd have some code that actually triggers your extension, and you can utilize the methods you wrote that calculate a suggested reviewer. Just some thoughts!

praiseA Nov. 10, 2018, 10:43 a.m.

Hey Storm! Thanks for the feedback. I have added the extension to the review request page. so when the model works, it will correctly show up on the review request page. Right now, I have a placeholder there with my username

Commit:

c080c8705c2967ed17f564e4f4b76fe369eebbd1

c8fdc405ea92939982249924393bea428e32dfed

Diff:

Revision 13 (+484)

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.
```
suggest_reviewers_extension/setup.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E113 unexpected indentation
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E501 line too long (90 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E501 line too long (99 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E113 unexpected indentation
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E501 line too long (82 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E271 multiple spaces after keyword
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E114 indentation is not a multiple of four (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E116 unexpected indentation (comment)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E501 line too long (99 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 13)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E401 multiple imports on one line
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 13)
The issue has been resolved. Show all issues
```
E225 missing whitespace around operator
```

Commit:

c8fdc405ea92939982249924393bea428e32dfed

db9a0baf05d98392f766aaa1f1e585fdda646f6a

Diff:

Revision 14 (+485)

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 40 failures.
```
suggest_reviewers_extension/setup.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E113 unexpected indentation
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (84 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
F841 local variable 'filename1' is assigned to but never used
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
F841 local variable 'filename2' is assigned to but never used
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (87 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (99 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (94 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (85 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 14)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```

Commit:

db9a0baf05d98392f766aaa1f1e585fdda646f6a

9caffb57738749a0acfb14ac779098365d278dd7

Diff:

Revision 15 (+495)

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

suggest_reviewers_extension/setup.py (Diff revision 15)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E271 multiple spaces after keyword
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
W503 line break before binary operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 15)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 15)
The issue has been resolved. Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```

Commit:

9caffb57738749a0acfb14ac779098365d278dd7

41fe94633963717c17734aa5905232d811855b08

Diff:

Revision 16 (+495)

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

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E271 multiple spaces after keyword
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
W503 line break before binary operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 16)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 16)
The issue has been resolved. Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```

Commit:

41fe94633963717c17734aa5905232d811855b08

ff7ae01b3ae853588a0fd51df1d7e0dba043141a

Diff:

Revision 17 (+508)

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

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ':'
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E271 multiple spaces after keyword
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W503 line break before binary operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 17)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 17)
The issue has been resolved. Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```

Commit:

ff7ae01b3ae853588a0fd51df1d7e0dba043141a

4befb7633f32fb472c6f0f0b7cc0a2820cbdd865

Diff:

Revision 18 (+499)

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

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ':'
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
F401 'pdb' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E303 too many blank lines (2)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E271 multiple spaces after keyword
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E303 too many blank lines (3)
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W503 line break before binary operator
```
suggest_reviewers_extension/suggest_reviewers_extension/test.py (Diff revision 18)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```
suggest_reviewers_extension/suggest_reviewers_extension/views.py (Diff revision 18)
The issue has been resolved. Show all issues
```
F401 'django.http.HttpResponse' imported but unused
```

suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 18)

The issue has been resolved. Show all issues

You're creating a "Model" here, but that's not really what you want. Django models correspond to database tables, and you're not storing things in the database. I think this can just be a normal class, and it probably would make sense to rename the file so as to not be confusing.

praiseA Nov. 19, 2018, 10:56 a.m.

thanks for pointing that out. i have changed it.

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 18)
The issue has been resolved. Show all issues
```
You should add a trailing comma over here
```
1. praiseA Nov. 19, 2018, 10:56 a.m.
  thanks for picking that

suggest_reviewers_extension/suggest_reviewers_extension/templates/suggest_reviewers_extension/template.html (Diff revision 18)

The issue has been resolved. Show all issues

You have some trailing whitespace over here

Commit:

4befb7633f32fb472c6f0f0b7cc0a2820cbdd865

012e3e30b530a22c1bfd6afeccc7b0ce5b63087a

Diff:

Revision 19 (+404)

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/SuggestReviewers.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 19)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 19)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 19)
The issue has been resolved. Show all issues
```
F401 'suggest_reviewers_extension.SuggestReviewersResource.suggestReviewersResource' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 19)
The issue has been resolved. Show all issues
```
F401 'djblets.webapi.resources.unregister_resource_for_model' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 19)
The issue has been resolved. Show all issues
```
F401 'djblets.webapi.resources.register_resource_for_model' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 19)
The issue has been resolved. Show all issues
```
E127 continuation line over-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 19)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ','
```

Commit:

012e3e30b530a22c1bfd6afeccc7b0ce5b63087a

c3035ca899ddb27e3e4536ac3b05f6a7583bdf69

Diff:

Revision 20 (+406)

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/SuggestReviewers.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 20)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 20)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ','
```

Commit:

c3035ca899ddb27e3e4536ac3b05f6a7583bdf69

7aea0bc11a5f1fe2840c5b95f80ced28bce545c2

Diff:

Revision 21 (+406)

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/SuggestReviewers.py
	5 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

reviewboard/reviews/models/countReviewers.py (Diff revision 21)

The issue has been dropped. Show all issues

Is it worth using a Counter here instead of a Dict? I don't know if this will actually boost performance in a noticeable way, but I know I've used it in the past for this kind of purpose. It can turn your algorithm below for key reviewer into a single method call as well.

praiseA Nov. 29, 2018, 3:35 p.m.

hey Jeff,

For this method, I need to count the reviews each different reviewer has done. So essentially it's a counter, but its for each reviewer. Does that make sense? or did I misunderstand your question?

reviewboard/reviews/models/retrieveInformation.py (Diff revision 21)

The issue has been resolved. Show all issues

I know that this is WIP so a lot of these comments could be self-notes so you can keep track of what you're doing, but I definitely think the majority of your comments could be done away with when it comes out of WIP.

praiseA Nov. 29, 2018, 3:35 p.m.
```
thanks Jeff, I'll keep that in mind!
```

reviewboard/reviews/models/retrieveInformation.py (Diff revision 21)

The issue has been dropped. Show all issues

Ditto some of your variable naming, some of them don't conform to standards and others are unclear/unspecific like download_dir being a filename, not a directory.

praiseA Nov. 29, 2018, 3:35 p.m.

you are right. I'll spend sometime going through my naming convention and make sure they are correct for Python. Thank you

reviewboard/reviews/models/reviewers.py (Diff revision 21)

The issue has been dropped. Show all issues

This seems to be hardcoded into a lot of these files - could it be centralized somewhere and retrieved instead of defined in each one?

praiseA Nov. 29, 2018, 3:35 p.m.

reviewers.py was majorly my testing script, when I was playing around with models. I wouldn't be submitting that file. Also retrieveInformation.py was the script I wrote to extract the public reviews from review board for testing my ML model's accuracy as well. I wouldn't be submitting that in the final patch as well.

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 21)

The issue has been resolved. Show all issues

I believe this docstring should end with a period, and the linebreaks look a bit off to me in the diff.

praiseA Nov. 29, 2018, 3:35 p.m.
```
I'll look into this! thanks
```

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 21)

The issue has been resolved. Show all issues

Is it worth checking that the files exist before this?

praiseA Nov. 29, 2018, 3:35 p.m.

Yeah you are right. I will check if file1 and file2 are not None first. Its better to be safe than sorry right? lol

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 21)

The issue has been dropped. Show all issues

It seems like Counter could work well here as well.

praiseA Nov. 29, 2018, 3:35 p.m.

hey, its the same as I said for the other question. I don't want a generic counter, I want it different for each reviewer because I want the reviewer with the highest count

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewers.py (Diff revision 21)

The issue has been dropped. Show all issues

Does an if not review_request.get_draft() fail here sometimes even when it's not None?

praiseA Nov. 29, 2018, 3:35 p.m.

I played around with this for a while, I don't think so. It's either the review_request is public and we can access it or its a draft and then we can get it that way

Commit:

7aea0bc11a5f1fe2840c5b95f80ced28bce545c2

6162e080ec777502b7ba603bc736873795bf89de

Diff:

Revision 22 (+552)

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/SuggestReviewersResource.py
	9 more

Checks run (2 failed)

flake8 failed.

JSHint failed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

E902 IndentationError: unindent does not match any outer indentation level

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

E127 continuation line over-indented for visual indent

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

W293 blank line contains whitespace

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

E302 expected 2 blank lines, found 1

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

W293 blank line contains whitespace

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 22)

The issue has been resolved. Show all issues

E999 IndentationError: unindent does not match any outer indentation level

suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 22)
The issue has been resolved. Show all issues
```
F401 'djblets.util.fields.JSONField' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 22)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 22)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/models.py (Diff revision 22)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```

JSHint

suggest_reviewers_extension/suggest_reviewers_extension/static/js/views/suggestReviewersView.js (Diff revision 22)

The issue has been resolved. Show all issues

Col: 9
 'self' is defined but never used.

Commit:

6162e080ec777502b7ba603bc736873795bf89de

231ad794a18f234122916664eded04a9d5a672dc

Diff:

Revision 23 (+553)

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/SuggestReviewersResource.py
	9 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 23)

The issue has been resolved. Show all issues

E231 missing whitespace after ','

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 23)

The issue has been resolved. Show all issues

E231 missing whitespace after ':'

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 23)

The issue has been resolved. Show all issues

E231 missing whitespace after ':'

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 23)

The issue has been resolved. Show all issues

E304 blank lines found after function decorator

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 23)

The issue has been resolved. Show all issues

E999 IndentationError: unexpected unindent

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E902 TokenError: EOF in multi-line statement
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E123 closing bracket does not match indentation of opening bracket's line
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E999 SyntaxError: invalid syntax
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E131 continuation line unaligned for hanging indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E123 closing bracket does not match indentation of opening bracket's line
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E113 unexpected indentation
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 23)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```

Commit:

231ad794a18f234122916664eded04a9d5a672dc

881857ff3572ef7b19212f556e3a7a15377e9a67

Diff:

Revision 24 (+552)

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/SuggestReviewersResource.py
	9 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 24)

The issue has been resolved. Show all issues

E999 IndentationError: unexpected unindent

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 24)

The issue has been resolved. Show all issues

E304 blank lines found after function decorator

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 24)

The issue has been resolved. Show all issues

W292 no newline at end of file

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 24)
The issue has been resolved. Show all issues
```
F401 'reviewboard.extensions.base.JSExtension' imported but unused
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 24)
The issue has been resolved. Show all issues
```
F821 undefined name 'SuggestReviewersJSExtension'
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 24)
The issue has been resolved. Show all issues
```
F821 undefined name 'SuggestReviewersJSExtension'
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 24)
The issue has been resolved. Show all issues
```
E128 continuation line under-indented for visual indent
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 24)
The issue has been resolved. Show all issues
```
F821 undefined name 'suggestReviewersResource'
```

Commit:

881857ff3572ef7b19212f556e3a7a15377e9a67

55d854a027b65e44e400fed7a282dd57722dd96b

Diff:

Revision 25 (+547)

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/SuggestReviewersResource.py
	9 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 25)

The issue has been resolved. Show all issues

E999 IndentationError: unexpected unindent

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 25)
The issue has been resolved. Show all issues
```
F821 undefined name 'suggest_reviewers_resource'
```

Commit:

55d854a027b65e44e400fed7a282dd57722dd96b

9bdc21ddf271eb17879915446b7fbdcba8a54d9b

Diff:

Revision 26 (+547)

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/SuggestReviewersResource.py
	9 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 26)

The issue has been resolved. Show all issues

E999 IndentationError: unexpected unindent

Commit:

9bdc21ddf271eb17879915446b7fbdcba8a54d9b

a1d7fd15b184bd78fbb13a3667cc4629cf6ce3a2

Diff:

Revision 27 (+537)

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/SuggestReviewersResource.py
	9 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

suggest_reviewers_extension/README.rst (Diff revision 27)
The issue has been resolved. Show all issues
```
Trailing whitespace here
```

Commit:

a1d7fd15b184bd78fbb13a3667cc4629cf6ce3a2

e28e6cc3d68767eec6e8a48b6025ae8db379659f

Diff:

Revision 28 (+745)

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/SuggestReviewersResource.py
	11 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 28)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 28)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
1. praiseA Dec. 8, 2018, 5:54 p.m.
  thanks!

Commit:

e28e6cc3d68767eec6e8a48b6025ae8db379659f

db5450449cb53c8def8d7a3bf2bd9f788970362c

Diff:

Revision 29 (+633)

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
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 29)
The issue has been resolved. Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 29)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 29)
The issue has been resolved. Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 29)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 29)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```

Commit:

db5450449cb53c8def8d7a3bf2bd9f788970362c

d732defd09868d36e1371ce7ebf4572dc9b022ee

Diff:

Revision 30 (+431)

Show changes

	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
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	4 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 30)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 30)
The issue has been resolved. Show all issues
```
E301 expected 1 blank line, found 0
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 30)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```
suggest_reviewers_extension/suggest_reviewers_extension/suggestReviewers.py (Diff revision 30)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```

Commit:

d732defd09868d36e1371ce7ebf4572dc9b022ee

db8c1a5e0869e7b87f50c3828fb9a0b1ce729d2b

Diff:

Revision 31 (+233)

Show changes

	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
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	3 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

reviewboard/reviews/models/countReviewers.py (Diff revision 31)

The issue has been resolved. Show all issues

This file is not a model. I'm confused as to its location (and why an extension would have RB models?)

praiseA Dec. 11, 2018, 10:40 p.m.

Hi Barret. I was using this script to test my first idea on how to suggest reviewers based on stats. It wouldn't be submitted. I would remove this script right away

reviewboard/reviews/models/countReviewers.py (Diff revision 31)

The issue has been resolved. Show all issues

This is a class, which isn't intended to contain arbitrary code.
In Java, everything must be contained in a class, but that is not the case for python. If you are intending for this to be a script you can remove the  class and keep everything at top-level.

praiseA Dec. 11, 2018, 10:40 p.m.

ohh okay, yes it was intended as a script, I will always keep that in mind from now on. Thanks!

reviewboard/reviews/models/retrieveInformation.py (Diff revision 31)

The issue has been resolved. Show all issues

This file is not a model. I'm confused as to its location (and why an extension would have RB models?)

praiseA Dec. 11, 2018, 10:40 p.m.

This script was used to retrieve review board reviews so I could use it to test the reliability of the ML model. I will remove this as well now.

reviewboard/reviews/models/reviewers.py (Diff revision 31)

The issue has been resolved. Show all issues

This file is not a model. I'm confused as to its location (and why an extension would have RB models?)

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 31)

The issue has been resolved. Show all issues

Docstrings should be of the format:

"""Single line summary.

Multi-line description.
"""

praiseA Dec. 11, 2018, 10:40 p.m.
```
thanks!
```

Commit:

db8c1a5e0869e7b87f50c3828fb9a0b1ce729d2b

bd9416670ba61d7e2ffe3a74e10ee820db6d6331

Diff:

Revision 32 (+433)

Show changes

	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/SuggestReviewers.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	4 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commit:

bd9416670ba61d7e2ffe3a74e10ee820db6d6331

2665e9b4de3b8de8ef34444ac21f518e1f09b865

Diff:

Revision 33 (+489)

Show changes

	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/SuggestReviewers.py
	suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

F401 'djblets.webapi.decorators.webapi_request_fields' imported but unused

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

F401 'reviewboard.webapi.decorators.webapi_login_required' imported but unused

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

F401 'reviewboard.webapi.decorators.webapi_check_local_site' imported but unused

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

F401 'reviewboard.webapi.resources.resources' imported but unused

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

E231 missing whitespace after ':'

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 33)

The issue has been resolved. Show all issues

E305 expected 2 blank lines after class or function definition, found 1

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 33)
The issue has been resolved. Show all issues
```
F401 'reviewboard.extensions.base.JSExtension' imported but unused
```

Commit:

2665e9b4de3b8de8ef34444ac21f518e1f09b865

3766656869d6df48f40ec467080b84c4ded97619

Diff:

Revision 34 (+376)

Show changes

	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/SuggestReviewers.py
	suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	3 more

Checks run (2 failed)

flake8 failed.

JSHint failed.

flake8

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 34)

The issue has been resolved. Show all issues

E501 line too long (80 > 79 characters)

suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py (Diff revision 34)

The issue has been resolved. Show all issues

E203 whitespace before ':'

suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 34)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers_extension/suggest_reviewers_extension/extension.py (Diff revision 34)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```

JSHint

suggest_reviewers_extension/suggest_reviewers_extension/static/js/suggestreviewersextension.es6.js (Diff revision 34)

The issue has been resolved. Show all issues

Col: 53
 Expected '!==' and instead saw '!='.

Commit:

3766656869d6df48f40ec467080b84c4ded97619

e7de5d5de14a5af637fba1932845b14e28c2e30c

Diff:

Revision 35 (+378)

Show changes

	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/SuggestReviewers.py
	suggest_reviewers_extension/suggest_reviewers_extension/SuggestReviewersResource.py
	suggest_reviewers_extension/suggest_reviewers_extension/__init__.py
	suggest_reviewers_extension/suggest_reviewers_extension/extension.py
	3 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commit:

e7de5d5de14a5af637fba1932845b14e28c2e30c

8e088bf50b0b46403ce63aa4ad461bedef602ddd

Diff:

Revision 36 (+365)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	2 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers/setup.py (Diff revision 36)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```

Commit:

8e088bf50b0b46403ce63aa4ad461bedef602ddd

fdfa9b2042f2acd3012d4c1b4837645e05bfe9d5

Diff:

Revision 37 (+365)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	2 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commit:

fdfa9b2042f2acd3012d4c1b4837645e05bfe9d5

15c175847366238bce3d5de904e867b85c01f557

Diff:

Revision 38 (+11585)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

suggest_reviewers/suggest_reviewers/SuggestReviewers.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E225 missing whitespace around operator
```
suggest_reviewers/suggest_reviewers/SuggestReviewers.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E122 continuation line missing indentation or outdented
```
suggest_reviewers/suggest_reviewers/SuggestReviewers.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
F401 'pdb' imported but unused
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ':'
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ':'
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
W291 trailing whitespace
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E226 missing whitespace around arithmetic operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E201 whitespace after '('
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E225 missing whitespace around operator
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E302 expected 2 blank lines, found 1
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
W293 blank line contains whitespace
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E231 missing whitespace after ','
```
suggest_reviewers/tests/SuggestReviewersAccuracy.py (Diff revision 38)
The issue has been resolved. Show all issues
```
E305 expected 2 blank lines after class or function definition, found 1
```
suggest_reviewers/tests/SuggestReviewersTests.py (Diff revision 38)
The issue has been resolved. Show all issues
```
W292 no newline at end of file
```

Commit:

15c175847366238bce3d5de904e867b85c01f557

557e4b39d0566922f7927e2000cb480297d8a975

Diff:

Revision 39 (+11593)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	6 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Summary:: [WIP]: Create an Extension that suggests reviewers
Create an Extension that suggests reviewers
Description:: ~
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.
~
~
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.
-
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.
Testing Done:: ~
Played around with Django models to extract reviewers
~
and the reviews that were made
~
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.
-
Created an extension folder and implemented the extension.py
-
Added views.py and models.py files

Commit:

557e4b39d0566922f7927e2000cb480297d8a975

778a475fe9987e2df52ac10b875797c53a9acaa3

Diff:

Revision 40 (+11579)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	5 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commit:

778a475fe9987e2df52ac10b875797c53a9acaa3

6f2aa0cc6395e589b8a95cd4202ec40e461800e0

Diff:

Revision 41 (+11579)

Show changes

	suggest_reviewers/MANIFEST.in
	suggest_reviewers/README.rst
	suggest_reviewers/setup.py
	suggest_reviewers/suggest_reviewers/SuggestReviewers.py
	suggest_reviewers/suggest_reviewers/SuggestReviewersResource.py
	suggest_reviewers/suggest_reviewers/__init__.py
	suggest_reviewers/suggest_reviewers/extension.py
	suggest_reviewers/suggest_reviewers/static/css/index.css
	5 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Status:: Discarded

~		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.

~		Played around with Django models to extract reviewers
~		and the reviews that were made
	~	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`.
-		Created an extension folder and implemented the extension.py
-		Added views.py and models.py files