-
-
Notifications
You must be signed in to change notification settings - Fork 448
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: update blas/ext/base/ssumors
to follow current project conventions
#1692
Conversation
lib/node_modules/@stdlib/blas/ext/base/ssumors/benchmark/benchmark.ndarray.native.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Athan <[email protected]>
Signed-off-by: Athan <[email protected]>
Signed-off-by: Athan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this, @lovelindhoni. If you can update the indentation of manifest.json
, I think this PR should be ready for merge.
Signed-off-by: Lovelin <[email protected]>
Signed-off-by: Philipp Burckhardt <[email protected]>
Let's get this in once CI passes; thanks for your contribution, @lovelindhoni! |
Apologies @Planeshifter, I was going to fix my silly indentation issue. Thanks for sparing your time. |
…nventions PR-URL: stdlib-js#1692 Closes: stdlib-js#1543 --------- Signed-off-by: Athan Reines <[email protected]> Signed-off-by: Lovelin <[email protected]> Signed-off-by: Philipp Burckhardt <[email protected]> Co-authored-by: Athan Reines <[email protected]> Co-authored-by: Philipp Burckhardt <[email protected]> Reviewed-by: Athan Reines <[email protected]> Reviewed-by: Philipp Burckhardt <[email protected]>
Resolves #1543 .
Description
This pull request refactors
blas/ext/base/ssumors
to follow current project conventions.Related Issues
Related issues #788 , #1152
This pull request:
blas/ext/base/ssumors
to follow current project conventions #1543Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers