sindresorhus / eslint-plugin-unicorn / 1422

98%

master: 99%

Pull #476

travis-ci

fix: invalid autofix with destructuring assignment in no-for-loops rule

Example (original):
```js
for (let i = 0; i < arr.length; i++) {
  const { a, b } = arr[i];
  console.log(a, b);
}
```

Autofix (before this fix):
```js
for (const element of arr) {
  console.log(a, b);
}
```

Autofix (after this fix):
```js
for (const element of arr) {
  const { a, b } = element;
  console.log(a, b);
}
```

When `arr[i]` is used in a destructuring assignment, the entire assignment should not be removed (previous behavior), only `arr[i]` should be replaced with `element` (new behavior).

3060 of 3202 branches covered (95.57%)

4274 of 4356 relevant lines covered (98.12%)

426.86 hits per line