compare current to sorted

Author: stevensacks

lib/index.js

@@ -18,6 +18,3 @@ const requireIndex = require('requireindex');
 
 // import all rules in lib/rules
 module.exports.rules = requireIndex(__dirname + '/rules');
-
-
-

lib/rules/sort-react-dependency-arrays.js

@@ -38,16 +38,19 @@ module.exports = {
                     const dependencies = node.arguments[1];
 
                     if (dependencies && dependencies.type === 'ArrayExpression' && dependencies.elements.length > 1) {
+                        const currentNames = dependencies.elements.map(item => item.name);
                         const sorted = [...dependencies.elements].sort((a, b) => (a.name < b.name ? -1 : 1));
                         const sortedNames = sorted.map((dep) => dep.name);
 
-                        context.report({
-                            node,
-                            message: 'Sorting dependencies',
-                            fix: function (fixer) {
-                                return fixer.replaceText(dependencies, `[${sortedNames}]`);
-                            }
-                        });
+                        if (String(currentNames) !== String(sortedNames)) {
+                            context.report({
+                                node,
+                                message: 'Sorting dependencies',
+                                fix: function (fixer) {
+                                    return fixer.replaceText(dependencies, `[${sortedNames}]`);
+                                }
+                            });
+                        }
                     }
                 }
             }

package.json

@@ -1,7 +1,7 @@
 {
     "name": "eslint-plugin-sort-react-dependency-arrays",
     "description": "ESLint plugin to alphabetically sort React hook dependency arrays",
-    "version": "0.0.1",
+    "version": "0.0.2",
     "author": "Steven Sacks <[email protected]>",
     "keywords": [
         "eslint",