- partially undoing lodash change

[ezolenko]

Partially undoing recent lodash change (#328), _.each() semantics is apparently different from Array.forEach() -- during the build it created "undefined" folder with bunch of empty files.

[agilgur5]

during the build it created "undefined" folder with bunch of empty files.

ah my bad, thanks for catching this and the exact line too.

I did see this folder once or twice but thought that was because of all the other changes and investigations I was doing and just a temporary thing I broke -- turns out it wasn't temporary 😅
It also doesn't pop up during testing because, well, there are no tests for this yet

_.each() semantics is apparently different from Array.forEach()

The semantics are only slightly different in that _.each handles falsey values, while .forEach doesn't -- but .forEach doesn't need to, as you can only use it on arrays anyway.

Another reason why I think it's a problem with this instead

[agilgur5]

See my suggested change to the line that causes this -- that's the only one that needs fixing and it's not a forEach issue, it's because of how I rewrote it (removing the anonymous arrow function). With that fix in, there should be no other issues with any lodash replacement.

So if we can just change only that line, think this would be good to go!

[ezolenko]

Right, I committed that change directly 4310873