Upgrade ESLint to 4.2.0

Author: robertrossmann

README.md

@@ -60,5 +60,5 @@ This software is licensed under the **BSD-3-Clause License**. See the [LICENSE](
 [eslint-url]: http://eslint.org
 [travis-badge]: https://travis-ci.org/strvcom/eslint-config-javascript.svg
 [travis-url]: https://travis-ci.org/strvcom/eslint-config-javascript
-[eslint-version]: https://img.shields.io/badge/ESLint->=4.0.0-brightgreen.svg
+[eslint-version]: https://img.shields.io/badge/ESLint->=4.2.0-brightgreen.svg
 [eslint-fixing]: http://eslint.org/docs/user-guide/command-line-interface#fix

coding-styles/recommended.js

@@ -79,6 +79,11 @@ module.exports = {
     // allowed on a single line can be very beneficial for readability and maintainability.
     'max-statements-per-line': 'warn',
 
+    // Enforce newlines between operands of ternary expressions
+    // Enforce either the whole ternary on a single line or each part of the ternary on its own line
+    // if the expression contains a newline
+    'multiline-ternary': ['warn', 'always-multiline'],
+
     // Require Constructors to Use Initial Caps
     // This rule is aimed at helping to distinguish regular functions from constructor functions. As
     // such, it warns whenever it sees new followed by an identifier that isn't capitalized or

environments/nodejs/recommended.js

@@ -53,6 +53,8 @@ module.exports = {
     // Disallow Synchronous Methods
     // This rule is aimed at preventing synchronous methods from being called in Node.js. It looks
     // specifically for the method suffix "Sync" (as is the convention with Node.js operations).
-    'no-sync': 'warn',
+    'no-sync': ['warn', {
+      allowAtRootLevel: true,
+    }],
   },
 }

environments/shared/recommended.js

@@ -58,6 +58,10 @@ module.exports = {
     // an infinite for loop is a bug.
     'for-direction': 'error',
 
+    // Enforce that a return statement is present in property getters
+    // Not returning a value from a property getter is most likely a programmer mistake.
+    'getter-return': 'error',
+
     // Require Guarding for-in
     // This rule is aimed at preventing unexpected behavior that could arise from using a for-in
     // loop without filtering the results in the loop.