Skip to content

Replace internal group-by.ts with native method #3856

Description

@ST-DDT

This is a reminder that groupBy is natively available in Node 22+.
So our workaround can be removed:

https://github.com/faker-js/faker/blob/next/src/internal/group-by.ts

We can do that when dropping support for Node 20 as part of v11.

Currently on hold, since we aren't ready for v11 yet.

Metadata

Metadata

Assignees

No one assigned

    Labels

    breaking changeCannot be merged when next version is not a major releasec: refactorPR that affects the runtime behavior, but doesn't add new features or fixes bugsp: 1-normalNothing urgents: on holdBlocked by something or frozen to avoid conflicts

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions