Update with airbnb/[email protected]

Minor change regarding quoted keywords. Rule is disabled on our style
guide so no differences from `HubSpot/[email protected]`.

Author: Nick Hwang

README.md

@@ -292,6 +292,28 @@ Other Style Guides (from Airbnb)
     };
     ```
 
+  - [3.8](#3.8) <a name="3.8"></a> Only quote properties that are invalid identifiers.
+
+  > Why? In general we consider it subjectively easier to read. It improves syntax highlighting, and is also more easily optimized by many JS engines.
+
+  eslint rules: [`quote-props`](http://eslint.org/docs/rules/quote-props.html).
+
+  ```javascript
+  // bad
+  const bad = {
+    'foo': 3,
+    'bar': 4,
+    'data-blah': 5,
+  };
+
+  // good
+  const good = {
+    foo: 3,
+    bar: 4,
+    'data-blah': 5,
+  };
+  ```
+
 **[⬆ back to top](#table-of-contents)**
 
 ## Arrays
@@ -361,8 +383,8 @@ Other Style Guides (from Airbnb)
     }
 
     // good
-    function getFullName(obj) {
-      const { firstName, lastName } = obj;
+    function getFullName(user) {
+      const { firstName, lastName } = user;
       return `${firstName} ${lastName}`;
     }
 
@@ -1891,6 +1913,8 @@ Other Style Guides (from Airbnb)
 
   - [21.3](#21.3) <a name='21.3'></a> Numbers: Use `Number` for type casting and `parseInt` always with a radix for parsing strings.
 
+    eslint rules: [`radix`](http://eslint.org/docs/rules/radix).
+
     ```javascript
     const inputValue = '4';
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hubspot-style",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "description": "HubSpot's version of a mostly reasonable approach to JavaScript",
   "scripts": {
     "difftool": "./bin/difftool",

packages/eslint-config-hubspot/CHANGELOG.md

@@ -1,3 +1,7 @@
+3.0.1 / 2016-01-06
+==================
+ - [fix] because we use babel, keywords should not be quoted
+
 3.0.0 / 2016-01-04
 ==================
  - [breaking] enable `quote-props` rule (#632)

packages/eslint-config-hubspot/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-hubspot",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "description": "HubSpot's ESLint config, following our styleguide",
   "main": "index.js",
   "scripts": {

packages/eslint-config-hubspot/rules/style.js

@@ -86,7 +86,7 @@ module.exports = {
     'padded-blocks': [2, 'never'],
     // require quotes around object literal property names
     // http://eslint.org/docs/rules/quote-props.html
-    'quote-props': [0, 'as-needed', { 'keywords': true, 'unnecessary': true, 'numbers': false }],
+    'quote-props': [0, 'as-needed', { 'keywords': false, 'unnecessary': true, 'numbers': false }],
     // specify whether double or single quotes should be used
     'quotes': [2, 'single', 'avoid-escape'],
     // require identifiers to match the provided regular expression