Comment by shawnz | Hacker News Reader

Top New Best Ask Show Jobs

Comment by shawnz | Hacker News Reader

Parent Full thread

It looks like they are also creating a "groupByToMap" for that reason. See: https://github.com/tc39/proposal-array-grouping

Property keys can also be symbol objects and it seems like that's supported by the groupBy function in addition to strings, see: https://tc39.es/proposal-array-grouping/

I wonder if groupBy should have had some sort of into argument. Otherwise, we may end up with four groupBy methods. groupBy, groupByToMap, groupByToWeakMap and groupByToRecord (should records land).

Edit: this has already been discussed https://github.com/tc39/proposal-array-grouping/issues/3

updated the post with groupByToMap, Thanks for pointing that out.

Ah fair enough!

Reply on news.ycombinator.com