Don't use .reverse()

sindresorhus · Jan 19, 2021 · 63f52ef · 63f52ef
1 parent c63e594
commit 63f52ef
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/rules/no-array-for-each.js b/rules/no-array-for-each.js
@@ -49,11 +49,11 @@ function getFixFunction(callExpression, sourceCode, functionInfo) {
 	const {returnStatements} = functionInfo.get(callback);
 
 	const getForOfLoopHeadText = () => {
-		const parametersText = parameters.map(parameter => sourceCode.getText(parameter));
+		const [elementText, indexText] = parameters.map(parameter => sourceCode.getText(parameter));
 		const useEntries = parameters.length === 2;
 
 		let text = 'for (const ';
-		text += useEntries ? `[${parametersText.reverse().join(', ')}]` : parametersText[0];
+		text += useEntries ? `[${indexText}, ${elementText}]` : elementText;
 
 		text += ' of ';