- add ESLint error-rule for prevent unnecessary console.log

Author: Valodya

.eslintrc

@@ -18,41 +18,37 @@
     "func-style": [
       "warn",
       "declaration",
-      { "allowArrowFunctions": true }
+      {
+        "allowArrowFunctions": true
+      }
     ],
-
     // disallow declaration of variables already declared in the outer scope
-   // "no-shadow": "warn",
+    // "no-shadow": "warn",
 
     // require let or const instead of var
     "no-var": "warn",
-
     "no-unused-vars": "warn",
-
+    "no-console": "error",
     "max-len": [
       "warn",
       120
     ],
-
     // require or disallow use of semicolons instead of ASI
     "semi": [
       "warn",
       "never"
     ],
-
     // require the use of === and !== except null comparison
     "eqeqeq": [
       "warn",
       "smart"
     ],
-
     // specify whether double or single quotes should be used
     "quotes": [
       "warn",
       "single",
       "avoid-escape"
     ],
-
     // require space before/after arrow function"s arrow
     "arrow-spacing": [
       "warn",
@@ -61,23 +57,21 @@
         "after": true
       }
     ],
-
-    "arrow-parens": ["warn", "as-needed"],
-
+    "arrow-parens": [
+      "warn",
+      "as-needed"
+    ],
     // suggest using of const declaration for variables that are never modified after declared
     "prefer-const": "warn",
-
     // restrict what can be thrown as an exception
     "no-throw-literal": "warn",
-
     // disallow Unused Expressions
     "no-unused-expressions": [
       "warn",
       {
         "allowShortCircuit": true
       }
     ],
-
     // require padding inside curly braces
     "object-curly-spacing": [
       "warn",

src/decorators/deprecated.js

@@ -5,6 +5,7 @@ export const deprecated = (namespace, alternative) => function(target, prop, des
     const mainMessage = `"${namespace}.${prop}" is deprecated and will be removed in the nearest release.`
     const helpMessage = `Please use "${alternative}" instead of.`
 
+    // eslint-disable-next-line no-console
     console.warn(`${mainMessage} ${alternative ? helpMessage : ''}`)
 
     return value.apply(this, arguments)

src/request/request.js

@@ -9,6 +9,7 @@ export const sendRequest = config => {
   //--TODO remove this checking when we get rid of all sync methods
   if (config.isAsync === false || !config.asyncHandler) {
     if (LocalVars.XMLHttpRequest) {
+      // eslint-disable-next-line no-console
       console.warn(
         'The sync methods of the Backendless API are deprecated and will be removed in the nearest future.\n' +
         'Please, use async methods instead.'

src/utils.js

@@ -165,6 +165,7 @@ const Utils = {
 
   synchronized(method) {
     return function() {
+      // eslint-disable-next-line no-console
       console.warn('Using of sync methods is an outdated approach. Please, use async methods.')
 
       const context = this