Be consistent with use of values() for collections

[nexushoratio]

The primary collections, Array, Set, and Map, all support iterating on the instance:

for (const for of bar) {
  ...
}

Unfortunately, the default iterator for Map is [key, value], as provided by the entries() method, while for both Array and Set, it is values().

Due to this inconsistency, it may make sense to always explicitly use the values() method.

But, such a policy would mean that every loop over an array would be a bit noisier with that method call, and it is a pretty common thing.

And there are also the DOM array-like objects NodeList, et al.

• Research external policies
• Experiment
• Decide on one
• Enforce via eslint if possible
• Migrate