Skip to content

Conversation

Neerajpathak07
Copy link
Member

Resolves #3175

Description

What is the purpose of this pull request?

This pull request:

  • add implementation for array/base/unary5d-by

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 21, 2024

Coverage Report

Package Statements Branches Functions Lines
array/base/unary5d-by $\color{green}181/181$
$\color{green}+100.00\%$
$\color{green}14/14$
$\color{green}+100.00\%$
$\color{green}1/1$
$\color{green}+100.00\%$
$\color{green}181/181$
$\color{green}+100.00\%$

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

@kgryte
Copy link
Member

kgryte commented Nov 21, 2024

@Neerajpathak07 You have a lint error which needs to be fixed in order to get CI passing.

@kgryte kgryte added Feature Issue or pull request for adding a new feature. Utilities Issue or pull request concerning general utilities. Needs Changes Pull request which needs changes before being merged. labels Nov 21, 2024
@kgryte kgryte changed the title feat: add implementation for array/base/unary5d-by feat: add array/base/unary5d-by Nov 21, 2024
@Neerajpathak07
Copy link
Member Author

@kgryte made the essential changes all the CI checks are passing!!

@kgryte kgryte added Needs Review A pull request which needs code review. and removed Needs Changes Pull request which needs changes before being merged. labels Dec 3, 2024
@kgryte
Copy link
Member

kgryte commented Dec 11, 2024

/stdlib merge

@stdlib-bot stdlib-bot added the bot: In Progress Pull request is currently awaiting automation. label Dec 11, 2024
@stdlib-bot stdlib-bot removed the bot: In Progress Pull request is currently awaiting automation. label Dec 11, 2024
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Dec 11, 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, @Neerajpathak07!

@kgryte kgryte merged commit 1788e0a into stdlib-js:develop Dec 11, 2024
12 checks passed
@Neerajpathak07 Neerajpathak07 deleted the unary5d-by branch December 16, 2024 19:36
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. Utilities Issue or pull request concerning general utilities.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFC]: add array/base/unary5d-by

4 participants