stdlib-js
feat: add `blas/base/icamax`
Progresses #2039
Description
What is the purpose of this pull request?
This pull request:
- includes the support for Javascript and obtains Fortran reference from this page
Related Issues
Does this pull request have any related issues?
This pull request:
- progresses #2039
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.
- [x] Read, understood, and followed the contributing guidelines.
@stdlib-js/reviewers
Coverage Report
| Package | Statements | Branches | Functions | Lines |
|---|---|---|---|---|
| blas/base/icamax | $\color{green}230/230$ $color{green}+100.00%$ |
$\color{green}15/15$ $color{green}+100.00%$ |
$\color{green}2/2$ $color{green}+100.00%$ |
$\color{green}230/230$ $color{green}+100.00%$ |
The above coverage report was generated for the changes in this PR.
Use Progresses in the description, Resolves will automatically close the issue associated with #2039 once this PR gets merged.
Use
Progressesin the description,Resolveswill automatically close the issue associated with #2039 once this PR gets merged.
Thank you Aman sir for making the necessary changes in this PR.
May I know is this PR ready for merge?
Hello @aman-095 sir, thank you for suggesting the changes and made the necessary changes. Please do check.
@ShabiShett07, before resolving any comment make sure the change is incorporated incase of any issues mention there itself. It helps later reviewing the PR again.
@aman-095, sorry for that mistake, I have checked the changes I made now, need your review