-
-
Notifications
You must be signed in to change notification settings - Fork 481
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/ssortins
to follow current project conventions
#2917
Conversation
blas/ext/base/ssortins
to follow current project conventionsblas/ext/base/ssortins
to follow current project conventions
@Planeshifter ready for initial review! |
this branch has a lot of garbage commits which i made while trying to figure things out, i can raise a new, clean PR from another branch if needed. |
@aayush0325 No reason to worry about the commits, we are squashing the commits anyway when merging PRs to ensure a clean git history. I just did a review and made a few changes, but overall your PR looked great! Thanks for your contribution! |
Thank you for you help and support! |
Resolves #1538 .
Description
This pull request:
blas/ext/base/ssortins
to follow current project conventions.blas/ext/base/ssortins
to follow current project conventions #1538 .Related Issues
This pull request:
blas/ext/base/ssortins
to follow current project conventions #1538Questions
No.
Other
Checklist
@stdlib-js/reviewers