eslint stylistic rules (#13177)

zaggino · ficristo · commit d00e46476f12 · 2017-03-13T20:27:39.000+01:00
* add some styling rules

* fix trailing commas
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -36,8 +36,17 @@ module.exports = {
         // https://eslint.org/docs/rules/#nodejs-and-commonjs
         "no-new-require": 2,
         // https://eslint.org/docs/rules/#stylistic-issues
+        "block-spacing": 1,
+        "brace-style": [1, "1tbs", { allowSingleLine: true }],
+        "camelcase": 1,
+        "comma-dangle": 2,
+        "comma-spacing": 1,
+        "comma-style": [1, "last"],
+        "computed-property-spacing": 1,
         "eol-last": 1,
+        "func-call-spacing": 1,
         "indent": [1, 4],
+        "key-spacing": [1, { beforeColon: false, afterColon: true }],
         "max-len": [1, 120],
         "new-cap": [0, {
             "capIsNewExceptions": [
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -129,7 +129,7 @@ module.exports = function (grunt) {
                             'keymap/{,*/}*',
                             'lib/{,*/}*',
                             'mode/{,*/}*',
-                            'theme/{,*/}*',
+                            'theme/{,*/}*'
                         ]
                     },
                     {
@@ -149,7 +149,7 @@ module.exports = function (grunt) {
                 force: true,
                 files: false
             },
-            src: ['dist/**/*'],
+            src: ['dist/**/*']
         },
         less: {
             dist: {
diff --git a/src/language/HTMLSimpleDOM.js b/src/language/HTMLSimpleDOM.js
@@ -82,7 +82,7 @@ define(function (require, exports, module) {
         rtc     : { rb: true, rt: true, rtc: true, rp: true },
         optgroup: { optgroup: true, option: true },
         option  : { option: true },
-        tbody   : { caption: true, colgroup: true, thead: true, tbody: true, tfoot: true, },
+        tbody   : { caption: true, colgroup: true, thead: true, tbody: true, tfoot: true },
         tfoot   : { caption: true, colgroup: true, thead: true, tbody: true },
         thead   : { caption: true, colgroup: true },
         tr      : { tr: true, th: true, td: true, caption: true },
diff --git a/src/nls/hr/strings.js b/src/nls/hr/strings.js
@@ -565,7 +565,7 @@ define({
     "COLLAPSE_ALL"                  : "Skupi Sve",
     "EXPAND_ALL"                    : "Proširi Sve",
     "COLLAPSE_CURRENT"              : "Skupi Trenutno",
-    "EXPAND_CURRENT"                : "Proširi Trenutno",
+    "EXPAND_CURRENT"                : "Proširi Trenutno"
 });
 
 /* Last translated for a57bd8888da0b44f382e591c64b394b065178277 */
diff --git a/src/preferences/PreferencesBase.js b/src/preferences/PreferencesBase.js
@@ -1937,7 +1937,7 @@ define(function (require, exports, module) {
          */
         getPrefixedSystem: function (prefix) {
             return new PrefixedPreferencesSystem(this, prefix + ".");
-        },
+        }
 
     });
 
diff --git a/src/project/FileTreeView.js b/src/project/FileTreeView.js
@@ -503,7 +503,7 @@ define(function (require, exports, module) {
                 },
                 DOM.ins({
                     className: "jstree-icon"
-                }),
+                })
             ];
 
             var thickness = _createThickness(this.props.depth);
diff --git a/src/utils/AnimationUtils.js b/src/utils/AnimationUtils.js
@@ -42,7 +42,7 @@ define(function (require, exports, module) {
             "OTransition"     : "oTransitionEnd",
             "MozTransition"   : "transitionend",
             "WebkitTransition": "webkitTransitionEnd",
-            "transition"      : "transitionend",
+            "transition"      : "transitionend"
         };
 
         _.forEach(transitions, function (value, key) {
diff --git a/test/spec/PreferencesBase-test.js b/test/spec/PreferencesBase-test.js
@@ -901,7 +901,7 @@ define(function (require, exports, module) {
                     "dontCloseTags": [],
                     "indentTags": [],
                     "whenClosing": true,
-                    "whenOpening": false,
+                    "whenOpening": false
                 });
             });
 
