Skip to content

Commit

Permalink
MDL-79003 eslint: Enable 'reportUnusedDisableDirectives' eslint setting
Browse files Browse the repository at this point in the history
This commit is a backport of MDL-74301 which should have been backported
at the time.

This will warn on unused /* eslint-disable */ statements.
  • Loading branch information
kabalin authored and andrewnicols committed Jan 30, 2024
1 parent deb801f commit 95458aa
Show file tree
Hide file tree
Showing 7 changed files with 5 additions and 7 deletions.
2 changes: 2 additions & 0 deletions .eslintrc
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
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

0 comments on commit 95458aa

Please sign in to comment.