JSHint
-
reviewboard/static/rb/js/utils/tests/keyBindingUtilsTests.es6.js (Diff revision 1)
Review Request #10455 — Created March 20, 2019 and submitted
This change converts all remaining test code to ES6.
Ran js-tests.
Summary | ID |
---|---|
087db3eb299015557b983157d04a6d638e8044b1 |
Description | From | Last Updated |
---|---|---|
Col: 19 'view' is defined but never used. |
reviewbot | |
Too many spaces. |
chipx86 | |
Col: 19 'view' is defined but never used. |
reviewbot | |
function() { |
chipx86 | |
function() It's better to let beforeEach(), it(), describe(), etc. be multi-line anyway so it's less of a change to add … |
chipx86 | |
Col: 13 Do not use 'new' for side effects. |
reviewbot |
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+2452 -2910) |
reviewboard/static/rb/js/utils/tests/keyBindingUtilsTests.es6.js (Diff revision 2) |
---|
Col: 19 'view' is defined but never used.
reviewboard/static/rb/js/reviewRequestPage/views/tests/reviewEntryViewTests.es6.js (Diff revision 2) |
---|
Too many spaces.
reviewboard/static/rb/js/views/tests/commentIssueBarViewTests.es6.js (Diff revision 2) |
---|
function()
It's better to let
beforeEach()
,it()
,describe()
, etc. be multi-line anyway so it's less of a change to add to them.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+2428 -2882) |
reviewboard/static/rb/js/utils/tests/keyBindingUtilsTests.es6.js (Diff revision 3) |
---|
Col: 13 Do not use 'new' for side effects.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+2428 -2882) |