JSHint
-
reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js (Diff revision 1) Show all issues
Review Request #9266 — Created Oct. 13, 2017 and discarded
wip don't look here
Description | From | Last Updated |
---|---|---|
Col: 10 Missing semicolon. |
![]() |
|
Col: 10 Missing semicolon. |
![]() |
|
Col: 10 Missing semicolon. |
![]() |
|
Col: 32 Expected '!==' and instead saw '!='. |
![]() |
|
Col: 13 Expected '{' and instead saw 'target'. |
![]() |
|
W291 trailing whitespace |
![]() |
|
E302 expected 2 blank lines, found 1 |
![]() |
|
E501 line too long (80 > 79 characters) |
![]() |
|
F841 local variable 'cell_classes' is assigned to but never used |
![]() |
|
F841 local variable 'cell_class_attr' is assigned to but never used |
![]() |
|
It looks like you've got some stuff from your easyfix branch leaking into this. |
|
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+351 -67) |
reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js (Diff revision 2) |
---|
Col: 10 Missing semicolon.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+387 -75) |
reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js (Diff revision 3) |
---|
Col: 10 Missing semicolon.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 4 (+387 -75) |
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 5 (+413 -85) |
reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js (Diff revision 5) |
---|
Col: 32 Expected '!==' and instead saw '!='.
reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js (Diff revision 5) |
---|
Col: 13 Expected '{' and instead saw 'target'.
Description: |
|
---|
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 6 (+522 -89) |
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 7 (+1091 -114) |
reviewboard/diffviewer/templatetags/difftags.py (Diff revision 7) |
---|
E302 expected 2 blank lines, found 1
reviewboard/diffviewer/templatetags/difftags.py (Diff revision 7) |
---|
E501 line too long (80 > 79 characters)
reviewboard/diffviewer/templatetags/difftags.py (Diff revision 7) |
---|
F841 local variable 'cell_classes' is assigned to but never used
reviewboard/diffviewer/templatetags/difftags.py (Diff revision 7) |
---|
F841 local variable 'cell_class_attr' is assigned to but never used
It looks like you've got a bunch of different changes all kind of mixed together. Is it possible to extract things so that you have one review request per change?
reviewboard/webapi/resources/review_request.py (Diff revision 7) |
---|
It looks like you've got some stuff from your easyfix branch leaking into this.