Skip to content

Conversation

aayush0325
Copy link
Member

Resolves #3150

Description

What is the purpose of this pull request?

This pull request:

  • adds map method to array/fixed-endian-factory
  • adds tests and benchmarks for the added function

Related Issues

Does this pull request have any related issues?

This pull request:

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.


@stdlib-js/reviewers

@stdlib-bot
Copy link
Contributor

stdlib-bot commented Nov 26, 2024

Coverage Report

Package Statements Branches Functions Lines
array/fixed-endian-factory $\color{red}848/1105$
$\color{green}+76.74\%$
$\color{red}62/71$
$\color{green}+87.32\%$
$\color{red}14/25$
$\color{green}+56.00\%$
$\color{red}848/1105$
$\color{green}+76.74\%$

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

@aayush0325
Copy link
Member Author

aayush0325 commented Nov 26, 2024

good evening! kindly give this a review @kgryte .

@kgryte kgryte added the Feature Issue or pull request for adding a new feature. label Nov 26, 2024
Copy link
Member

@kgryte kgryte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks, @aayush0325.

@kgryte kgryte merged commit 1d217ba into stdlib-js:develop Nov 27, 2024
11 checks passed
@aayush0325 aayush0325 deleted the array-map branch February 16, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Issue or pull request for adding a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFC]: add map method to array/fixed-endian-factory

3 participants