Merge pull request #105 from mobify/release-v2.4.3

Release v2.4.3: Trailing Commas

Author: Marlow Payne

CHANGELOG

@@ -1,3 +1,5 @@
+2.4.3
+    - Add eslint rule + doc entry regarding trailing commas
 2.4.2
     - Add/update documentation for Sass-Lint
 2.4.1

javascript/.eslintrc

@@ -110,6 +110,9 @@
         "semi": 2,
 
         // Disallow function definitions of the form 'function myFunc() {'
-        "func-style": [2, "expression"]
+        "func-style": [2, "expression"],
+
+        // Enforce trailing comma style
+        "comma-style": [2, "last"]
     }
 }

javascript/README.md

@@ -105,6 +105,32 @@ a = b;
 (f());
 ```
 
+##Use trailing commas
+
+For multi-line lists or object properties place the commas separating items at the end of the line of the previous item.
+
+```javascript
+// bad
+var foo = ["apples"
+           , "oranges"
+           , "bananas"];
+
+var foo = {
+    "fruit": "apple"
+    , "vegetable": "arugula"
+};
+
+// good
+var foo = ["apples",
+           "oranges",
+           "bananas"];
+
+var foo = {
+    "fruit": "apple",
+    "vegetable": "arugula"
+};
+```
+
 ##Use function expressions over function declarations
 
 The function expression is clearly recognisable as what it really is (a variable with a function value). Additionally, it helps organize code so that all variable declarations appear at the top of a file, and invocations follow. This gives some predictablity when others are reading your code, allowing for a more consistent structure.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mobify-code-style",
-  "version": "2.4.2",
+  "version": "2.4.3",
   "description": "Code style guide and linting tools for Mobify",
   "main": "index.js",
   "scripts": {