refactor: improve type declarations by using generics in utils/group

[rajutkarsh07]

Resolves #1086

Description

What is the purpose of this pull request?

This pull request:

• Adds generics for improved type safety for the group function.

Related Issues

Does this pull request have any related issues?

This pull request:

• resolves [RFC]: improve type declarations for @stdlib/utils/group #1086

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

• Read, understood, and followed the contributing guidelines.

---

@stdlib-js/reviewers

[kgryte]

@Planeshifter Would you mind reviewing? You might have additional ideas regarding the keys of the output object.

[stdlib-bot]

👋 Hi there! 👋

And thank you for opening your first pull request! We will review it shortly. 🏃 💨

[Planeshifter]

The type of the collection elements does not have to match the types of the group elements. For example, we could have a numeric array and a group array of strings denoting the group labels. So we shouldn't use the T generic for both collections.

In fact, I suggest we enforce that the groups collection groups should contain only values that are valid property keys

[rajutkarsh07]

@Planeshifter I have updated the code

[Planeshifter]

Two suggestions, then this should be ready to land; thanks!

[rajutkarsh07]

@Planeshifter I have made the required changes

[Planeshifter]

@rajutkarsh07 Great, thanks! Will merge after CI has passed.