Files: * Consistently have one (1) new line at EOF (.gitignore, *.css, *.sh) JSHint: * Separate the common (general Wikimedia code conventions) from the local more stricter rules (nice work!). Maintenance: * Use simplified $IP definition where getenv is only called once. Based on http://git.io/ve-mw-36fc3b59-makeStaticLoader JavaScript: * Use the same closure in all files: - No need for "undefined". - One linebreak between license header and start of closure (sometimes there were 1 or 2 empty lines). - ext.uls.ime.js: No need for "document" (redundant in general, but especially as it wasn't used anywhere in this file). - Use "$, mw" for all ext.* modules (one of them used "mw, $"). Change-Id: Iba5dab14bbc3fbfc9f8fe96dd2d95529587a69aa
33 lines
452 B
Plaintext
33 lines
452 B
Plaintext
{
|
|
|
|
"camelcase": true,
|
|
"curly": true,
|
|
"eqeqeq": true,
|
|
"immed": true,
|
|
"latedef": true,
|
|
"newcap": true,
|
|
"noarg": true,
|
|
"noempty": true,
|
|
"nonew": true,
|
|
"quotmark": "single",
|
|
"trailing": true,
|
|
"undef": true,
|
|
"unused": true,
|
|
|
|
"bitwise": true,
|
|
"forin": false,
|
|
"regexp": false,
|
|
"strict": true,
|
|
"laxbreak": true,
|
|
"smarttabs": true,
|
|
"multistr": true,
|
|
"browser": true,
|
|
"onevar": true,
|
|
|
|
"predef": [
|
|
"mediaWiki",
|
|
"jQuery",
|
|
"QUnit"
|
|
]
|
|
}
|