From 8148bfce3c670a8a234ee71903ca020e32506884 Mon Sep 17 00:00:00 2001 From: Vladimir Kutepov Date: Tue, 10 Oct 2017 18:21:50 +0300 Subject: [PATCH] [eslint config] [base] [patch] Do not prefer destructuring for object assignment expression ```js let param = 'default value'; if (object) param = object.param; // <= prefer-destructuring } // ... ``` I think that `param = object.param;` is better than `({ param } = object);` because it is easier to read and tedious to write brackets by hand. MDN: [Destructuring assignment](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment) --- packages/eslint-config-airbnb-base/rules/es6.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/eslint-config-airbnb-base/rules/es6.js b/packages/eslint-config-airbnb-base/rules/es6.js index dac5f4b98f..bfdc6d29e4 100644 --- a/packages/eslint-config-airbnb-base/rules/es6.js +++ b/packages/eslint-config-airbnb-base/rules/es6.js @@ -118,7 +118,7 @@ module.exports = { }, AssignmentExpression: { array: true, - object: true, + object: false, }, }, { enforceForRenamedProperties: false,