eslint stylistic rules (adobe#13177)

* add some styling rules * fix trailing commas
Kriv-Art · Mar 13, 2017 · d00e464 · d00e464
1 parent 8561a11
commit d00e464
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 8 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -36,8 +36,17 @@ module.exports = {
         // http://eslint.org/docs/rules/#nodejs-and-commonjs
         "no-new-require": 2,
         // http://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
                 });
             });
-Original file line number
+Diff line change
@@ Expand Up / @@ -1937,7 +1937,7 @@ define(function (require, exports, module) { @@
              */
             getPrefixedSystem: function (prefix) {
                 return new PrefixedPreferencesSystem(this, prefix + ".");
-            },
+            }
         });
@@ Expand Down @@