Merge pull request #43 from htmlacademy/feature/add-rules-with-eslint-recommended

Added  eslint:recommended config

Author: Sasha Sushko

README.md

@@ -61,5 +61,25 @@ For validating **ECMAScript 2015 (ES6)** in **Node.js** environment project use
   }
 }
 ```
+
+For validating **ECMAScript 2018 (ES9)** project use `es9` version:
+
+```json
+{
+  "parserOptions": {
+    "ecmaVersion": 9,
+    "sourceType": "module"
+  },
+  "env": {
+    "es2017": true,
+    "browser": true,
+    "commonjs": true,
+  },
+  "extends": "htmlacademy/basic",
+  "rules": {
+    // Additional rules...
+  }
+}
+```
 [travis-image]: https://travis-ci.org/htmlacademy/eslint-config-htmlacademy.svg?branch=master
 [travis-url]: https://travis-ci.org/htmlacademy/eslint-config-htmlacademy

basic.js

@@ -0,0 +1,25 @@
+module.exports = {
+  extends: 'eslint:recommended',
+  rules: {
+    // Possible Errors
+    // https://eslint.org/docs/rules/#possible-errors
+    // ---------------------------------------------
+    'no-console': 'error',
+    // Stylistic Issues
+    // https://eslint.org/docs/rules/#stylistic-issues
+    // ---------------------------------------------
+    'comma-dangle': ['error', {
+      'arrays': 'always-multiline',
+      'objects': 'always-multiline',
+      'functions': 'always-multiline',
+    }],
+    'indent': ['error', 2, {
+      SwitchCase: 1,
+    }],
+    'quotes': ['error', 'single'],
+    // ECMAScript 6
+    // https://eslint.org/docs/rules/#ecmascript-6
+    // ---------------------------------------------
+    'no-var': ['error'],
+  },
+};

test/basic/.eslintrc.js

@@ -0,0 +1,13 @@
+const config = require('../../basic');
+
+config.parserOptions = {
+  ecmaVersion: 9,
+  sourceType: 'module',
+};
+config.env = {
+  'es2017': true,
+  'browser': true,
+  'commonjs': true,
+};
+
+module.exports = config;

test/basic/comma-dangles.js

@@ -0,0 +1,29 @@
+const arr0 = ['a', 'b', 'c'];
+const arr1 = [
+  'a',
+  'b',
+  'c',
+];
+
+const obj0 = {q: 'q', w: 'w'};
+const obj1 = {
+  q: 'q',
+  w: 'w',
+};
+
+const func0 = (a, b) => {
+  return a + b;
+};
+const func1 = (
+  a,
+  b,
+) => {
+  return a + b;
+};
+
+func0(
+  obj0.a,
+  obj0.b,
+);
+
+throw new Error(arr0, arr1, obj0, obj1, func0, func1);

test/basic/eslint.js

@@ -0,0 +1,25 @@
+module.exports = {
+  extends: 'eslint:recommended',
+  rules: {
+    // Possible Errors
+    // https://eslint.org/docs/rules/#possible-errors
+    // ---------------------------------------------
+    'no-console': 'error',
+    // Stylistic Issues
+    // https://eslint.org/docs/rules/#stylistic-issues
+    // ---------------------------------------------
+    'comma-dangle': ['error', {
+      'arrays': 'always-multiline',
+      'objects': 'always-multiline',
+      'functions': 'always-multiline',
+    }],
+    'indent': ['error', 2, {
+      SwitchCase: 1,
+    }],
+    'quotes': ['error', 'single'],
+    // ECMAScript 6
+    // https://eslint.org/docs/rules/#ecmascript-6
+    // ---------------------------------------------
+    'no-var': ['error'],
+  },
+};

test/basic/indent.js

@@ -0,0 +1,50 @@
+'use strict';
+
+// Continuation, aka MemberExpression
+const promise = window.Promise.resolve(true);
+promise.
+  then(function (data) {
+    return data;
+  }).
+  then(function (truthy) {
+    return !truthy;
+  }).
+  catch(function () {
+    return false;
+  });
+
+// Function expression
+const fun = function (first, second) {
+  return first + second;
+};
+
+// Function declaration
+function Constructor(first, second) {
+  this.data = {
+    first: first,
+    second: second,
+  };
+}
+
+// Calling site arguments
+const myObject = new Constructor(
+  'Petya',
+  'Vasya',
+);
+myObject.toString();
+
+const result = fun(
+  'one',
+  'two',
+);
+result.toString();
+
+// Switch
+const a = 'a';
+
+switch(a) {
+  case 'a':
+    break;
+  case 'b':
+    break;
+}

test/basic/no-var.js

@@ -0,0 +1,4 @@
+let a = 0;
+const B = 'B';
+
+throw new Error (a, B);

test/basic/quotes.js

@@ -0,0 +1,6 @@
+const a = 'Anything string';
+const b = `Anything
+string`;
+const c = `This's a ${a}`;
+
+throw new Error(a, b, c);