mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/RelatedArticles
synced 2024-11-24 08:14:21 +00:00
529b798198
* Remove file pattern whitelist for JSHint and JSCS. Instead, use the ignore list (which we need to specify anyway to avoid errors when using IDE plugins and/or running these from the command-line directly. * Remove redundant '*.json' pattern for banana. '**/*.json' covers this already as '**' is recursive from the current directory, otherwise it would be '*/**'. * Remove redundant 'name' and 'version' property. These were likely added to suppress warnings by npm-install. However those warnings are intended for packages published to npmjs.org. For local build tools, adding private:true also suppress these warnings. For extensions, name and versions are already maintained in other files such as extension.json and others. * Remove redundant '$' entry in jshintrc. There's no need to tolerate use of global variable '$'. The convention is to use 'jQuery' and alias it locally, which this extension already does everywhere. * Set 'jQuery' to false instead of true in jshint globals. True means it allows and expects this package to assign or expose the 'jQuery' identifier. However that is not the case. If this code overwrites jQuery, that should probably trigger a warning, thus set to false, which means read-only global. Change-Id: I0b159b6d684f67933e0dae506db1eb3800a6f192
22 lines
269 B
Plaintext
22 lines
269 B
Plaintext
{
|
|
// Enforcing
|
|
"bitwise": true,
|
|
"eqeqeq": true,
|
|
"es3": true,
|
|
"latedef": true,
|
|
"noarg": true,
|
|
"nonew": true,
|
|
"undef": true,
|
|
"unused": true,
|
|
|
|
// Environment
|
|
"browser": true,
|
|
|
|
"globals": {
|
|
"mw": false,
|
|
"jQuery": false,
|
|
"OO": false,
|
|
"QUnit": false
|
|
}
|
|
}
|