feat: add toLocaleString method to array/fixed-endian-factory

[ahmad-kashkoush]

Resolves #3158.

Description

What is the purpose of this pull request?

This pull request:

• adds support for a toLocaleString method to @stdlib/array/fixed-endian-factory.

Related Issues

Does this pull request have any related issues?

This pull request:

• resolves [RFC]: add toLocaleString method to array/fixed-endian-factory #3158

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

[stdlib-bot]

Coverage Report

Package	Statements	Branches	Functions	Lines
array/fixed-endian-factory	$\color{red}1179/1431$ $\color{green}+82.39\%$	$\color{red}156/169$ $\color{green}+92.31\%$	$\color{red}23/33$ $\color{green}+69.70\%$	$\color{red}1179/1431$ $\color{green}+82.39\%$

The above coverage report was generated for the changes in this PR.

[ahmad-kashkoush]

Good evening, @kgryte may you give this a review

[kgryte]

Thanks for working on this, @ahmad-kashkoush. This LGTM. If you wouldn't mind resolving the merge conflicts, that would be appreciated. Once resolved, should be ready for merge.

[ahmad-kashkoush]

Thanks for working on this, @ahmad-kashkoush. This LGTM. If you wouldn't mind resolving the merge conflicts, that would be appreciated. Once resolved, should be ready for merge.

@kgryte resolved conflicts, may you check it again?

[Planeshifter]

@kgryte @ahmad-kashkoush Looks like the benchmarks here are very slow; this might warrant a look. At the minimum, should adjust the benchmark files so we don't have CI running for ~20min to run the benchmarks for this method.

[ahmad-kashkoush]

@kgryte @ahmad-kashkoush Looks like the benchmarks here are very slow; this might warrant a look. At the minimum, should adjust the benchmark files so we don't have CI running for ~20min to run the benchmarks for this method.

Working on it 🫡

[ahmad-kashkoush]

@Planeshifter Can I reduce the benchmark scope?
the method itself runs in O(N) so can't be updated more to my knowledge
or may you suggest a better change I can do

[ahmad-kashkoush]

@Planeshifter May you reply to that comment #3311 (comment),
So that I can continue working on it?