Bug 1553451 - Lower the eslint complexity level to 20 for devtools and exlude more complex functions; r=jdescottes

We want to go back to ESLint's default complexity level so that newly introduced code is checked for complexity.
At the same time, to make that work, we're excluding all of the more complex functions for now.
We should fix them: make them less complex, and remove the eslint-disable comment.
See bug 1553449 for more information about this.

Differential Revision: https://phabricator.services.mozilla.com/D32139
This commit is contained in:
Patrick Brosset
2019-05-23 07:07:49 +00:00
parent 41544774b1
commit 18bd75fb8c
48 changed files with 128 additions and 2 deletions

View File

@@ -270,6 +270,7 @@ define(function(require, exports, module) {
// Event Handlers
/* eslint-disable complexity */
onKeyDown(event) {
if (!SUPPORTED_KEYS.includes(event.key)) {
return;
@@ -352,6 +353,7 @@ define(function(require, exports, module) {
this.treeRef.current.focus();
event.preventDefault();
}
/* eslint-enable complexity */
onClickRow(nodePath, event) {
const onClickRow = this.props.onClickRow;