Релиз 5.0.0 (#73)

* Запретит присваивание в return (#65) https://app.clickup.com/t/rgc43n
* Увеличит версию поддерживаемого стандарта ECMAScript (#63)
* Добавит правило, запрещаюшее писать выражения без смысла (#64) https://app.clickup.com/t/n183gm
* Удалит право на обязательную висячую запятую (#66) https://app.clickup.com/t/n1c2my
* Разрешит не разделять пустой строкой однострочные свойства (#72)

Co-authored-by: Lydia Zakharova <jahontis@gmail.com>

Author: Sasha Sushko

README.md

@@ -19,7 +19,7 @@ For validating **Vanilla JS** project use `vanilla` version:
 ```json
 {
   "parserOptions": {
-    "ecmaVersion": 2018,
+    "ecmaVersion": 2019,
     "sourceType": "module"
   },
   "env": {
@@ -38,7 +38,7 @@ For validating **React** project use `react` version (`htmlacademy/react` includ
 ```json
 {
   "parserOptions": {
-    "ecmaVersion": 2018,
+    "ecmaVersion": 2019,
     "sourceType": "module"
   },
   "env": {

changelog.md

@@ -1,3 +1,12 @@
+5.0.0 / 2021-11-17
+==================
+
+  * Убрали запрет и требование висячей запятой. Теперь она по желанию
+  * Подняли версию ES до 2019
+  * Добавили новые правила
+    * `no-unused-expressions`
+    * `no-return-assign`
+
 4.1.0 / 2021-09-29
 ==================
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-htmlacademy",
-  "version": "4.1.0",
+  "version": "5.0.0",
   "description": "ESLint shareable config for the HTML Academy style",
   "files": [
     "es5.js",

package.json

@@ -2,7 +2,7 @@
 const config = require('../../vanilla');
 
 config.parserOptions = {
-  ecmaVersion: 2018,
+  ecmaVersion: 2019,
   sourceType: 'module',
 };
 

test/vanilla/.eslintrc.js

@@ -1,13 +1,13 @@
-() => {};
+const fn1 = () => {};
 
-(anything) => anything;
+const fn2 = (anything) => anything;
 
-(anything) => {
+const fn3 = (anything) => {
   anything++;
   return anything;
 };
 
-(anything) => `${anything} \n`;
+const fn4 = (anything) => `${anything} \n`;
 
 const emptyString = '';
 
@@ -21,3 +21,5 @@ emptyString.then((anything) => {
     return anything;
   }
 });
+
+throw new Error(fn1, fn2, fn3, fn4);

test/vanilla/arrow-parens.js

@@ -1,3 +1,5 @@
+/* eslint-disable no-unused-expressions */
+
 const conditionLeft = 'str';
 const conditionRight = 'str';
 

test/vanilla/comma-dangles.js

@@ -25,11 +25,11 @@ doEnything(function() {
 });
 
 doEnything(function() {
-  this.key;
+  doEnything(this.key);
 });
 
 doEnything(function() {
-  (() => this);
+  (() => this)();
 });
 
 const someObject = {};

test/vanilla/eqeqeq.js

@@ -14,6 +14,8 @@ module.exports = {
     'no-alert': 'error',
     'no-useless-concat': 'error',
     'no-useless-return': 'error',
+    'no-unused-expressions': 'error',
+    'no-return-assign': 'error',
     'radix': 'error',
     // Strict Mode
     // https://eslint.org/docs/rules/#strict-mode
@@ -26,16 +28,11 @@ module.exports = {
     // https://eslint.org/docs/rules/#stylistic-issues
     // ---------------------------------------------
     'camelcase': 'error',
-    'comma-dangle': ['error', {
-      'arrays': 'always-multiline',
-      'objects': 'always-multiline',
-      'functions': 'always-multiline',
-    }],
     'eol-last': 'error',
     'indent': ['error', 2, {
       SwitchCase: 1,
     }],
-    'lines-between-class-members': ['error', 'always'],
+    'lines-between-class-members': ['error', 'always', {'exceptAfterSingleLine': true}],
     'no-multiple-empty-lines': 'error',
     'no-nested-ternary': 'error',
     'no-trailing-spaces': 'error',