Skip to content

Commit

Permalink
MDL-74301 eslint: Enable 'reportUnusedDisableDirectives' eslint setting
Browse files Browse the repository at this point in the history
This will warn on unused /* eslint-disable */ statements.
  • Loading branch information
kabalin committed Jul 3, 2023
1 parent 58d8a12 commit 572ca40
Show file tree
Hide file tree
Showing 7 changed files with 5 additions and 7 deletions.
2 changes: 2 additions & 0 deletions .eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@
'M': true,
'Y': true
},
// Warn about unused eslint-disable statements.
'reportUnusedDisableDirectives': true,
'rules': {
// See http://eslint.org/docs/rules/ for all rules and explanations of all
// rules.
Expand Down
2 changes: 1 addition & 1 deletion admin/tool/lp/amd/build/competency_rule.min.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions admin/tool/lp/amd/src/competency_rule.js
Original file line number Diff line number Diff line change
Expand Up @@ -71,14 +71,12 @@ define(['jquery'], function($) {
return null;
};

// eslint-disable-line valid-jsdoc
/**
* Return the type of the module.
*
* @return {String}
* @method getType
*/
// eslint-enable-line valid-jsdoc
Rule.prototype.getType = function() {
throw new Error('Not implemented');
};
Expand Down
Loading

0 comments on commit 572ca40

Please sign in to comment.