@putout/plugin-conditions 1.0.1 → 1.0.3

package/README.md

@@ -32,7 +32,7 @@ npm i @putout/plugin-conditions -D
 > The result of evaluating an equality operator is always of type boolean based on whether the comparison is true.
 >
 > (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators)
- 
+
 Checkout it 🐊[Putout Editor](https://putout.cloudcmd.io/#/gist/c61c94d2e1990f59b160aaf462f9a903/0855844b114079ec46098be6f3602dfdaa74290c).
 
 ### ❌ Example of incorrect code
@@ -61,8 +61,10 @@ Linter | Rule | Fix
 ### ❌ Example of incorrect code
 
 ```js
-if (2 > 3);
-    alert();
+if (2 > 3)
+    ;
+
+alert();
 ```
 
 ### ✅ Example of correct code
@@ -77,6 +79,7 @@ if (2 > 3)
 > Strict equality compares two values for equality. Neither value is implicitly converted to some other value before being compared. If the values have different types, the values are considered unequal.
 >
 > (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Equality_comparisons_and_sameness)
+
 ## ❌ Example of incorrect code
 
 ```js
@@ -98,15 +101,13 @@ const t = false;
 ### ❌ Example of incorrect code
 
 ```js
-if (a == b) {
-}
+if (a == b) {}
 ```
 
 ### ✅ Example of correct code
 
 ```js
-if (a === b) {
-}
+if (a === b) {}
 ```
 
 ## evaluate
@@ -183,8 +184,7 @@ if (zone?.tooltipCallback) {
 }
 
 if (a)
-    alert('hello');
-else
+    alert('hello');else
     alert('hello');
 ```
 

package/lib/apply-if/index.js

@@ -10,10 +10,14 @@ module.exports.filter = (path) => {
     if (!nextPath.node)
         return false;
     
-    return path.get('consequent').isEmptyStatement();
+    return path
+        .get('consequent')
+        .isEmptyStatement();
 };
 
-module.exports.include = () => ['IfStatement'];
+module.exports.include = () => [
+    'IfStatement',
+];
 
 module.exports.fix = (path) => {
     const nextPath = path.getNextSibling();

package/lib/convert-comparison-to-boolean/index.js

@@ -5,7 +5,11 @@ const {
     operator,
 } = require('putout');
 
-const {replaceWith, compute} = operator;
+const {
+    replaceWith,
+    compute,
+} = operator;
+
 const {
     isIdentifier,
     BooleanLiteral,

package/lib/convert-equal-to-strict-equal/index.js

@@ -11,4 +11,3 @@ module.exports.replace = () => ({
     '__a == __b': '__a === __b',
     '__a != __b': '__a !== __b',
 });
-

package/lib/evaluate/index.js

@@ -22,4 +22,3 @@ module.exports.replace = () => ({
         return '__b';
     },
 });
-

package/lib/index.js

@@ -13,4 +13,3 @@ module.exports.rules = {
     ...getRule('remove-boolean'),
     ...getRule('simplify'),
 };
-

package/lib/remove-boolean/index.js

@@ -5,19 +5,14 @@ module.exports.report = () => 'Avoid boolean in assertions';
 module.exports.replace = () => ({
     'return __a === true': 'return Boolean(__a)',
     'return __a == true': 'return Boolean(__a)',
-    
-    'const __a = __b === true': 'const __a = __b === Boolean(__a)',
-    'const __a = __b == true': 'const __a = __b == Boolean(__a)',
-    
+    'const __a = __b === true': 'const __a = __b',
+    'const __a = __b == true': 'const __a = __b',
     '__a = __b === true': '__a = __b === Boolean(__a)',
     '__a = __b == true': '__a = __b == Boolean(__a)',
-    
     '__a === true': '__a',
     '__a === false': '!__a',
     '__a == true': '__a',
     '__a == false': '!__a',
-    
     '__a !== true': '!__a',
     '__a != true': '!__a',
 });
-

package/lib/remove-constant/index.js

@@ -12,6 +12,7 @@ const {
     replaceWithMultiple,
     remove,
 } = operator;
+
 const {isIdentifier} = types;
 
 module.exports.report = () => 'Avoid constant conditions';
@@ -22,9 +23,7 @@ module.exports.fix = ({path, result}) => {
         consequent,
     } = path.node;
     
-    const {
-        body = [consequent],
-    } = consequent;
+    const {body = [consequent]} = consequent;
     
     if (result)
         return replaceWithMultiple(path, body);
@@ -38,6 +37,7 @@ module.exports.fix = ({path, result}) => {
 module.exports.traverse = ({push, generate}) => ({
     IfStatement(path) {
         const testPath = path.get('test');
+        
         const {
             left,
             right,
@@ -78,4 +78,3 @@ function containsIdentifiers(testPath) {
     
     return is;
 }
-

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@putout/plugin-conditions",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds support of conditions transformations",
-  "homepage": "https://github.com/coderaiser/conditions/tree/master/packages/plugin-conditions#readme",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-conditions#readme",
   "main": "lib/index.js",
   "release": false,
   "tag": false,
@@ -32,13 +32,12 @@
     "conditions"
   ],
   "devDependencies": {
-    "@putout/plugin-convert-for-each-to-for-of": "^8.1.0",
-    "@putout/plugin-remove-useless-variables": "^7.3.0",
+    "@putout/plugin-for-of": "*",
     "@putout/test": "^6.0.0",
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-n": "^15.2.4",
-    "eslint-plugin-putout": "^16.0.0",
+    "eslint-plugin-putout": "^17.0.0",
     "lerna": "^6.0.1",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",