feat: add array/base/with#1374
Conversation
updates value at a particular index Fixes stdlib-js#1328
stdlib-bot
left a comment
There was a problem hiding this comment.
👋 Hi there! 👋
And thank you for opening your first pull request! We will review it shortly. 🏃 💨
There was a problem hiding this comment.
Thank you for working on this; made a few changes and left a bunch of comments that will have to be addressed, but overall this is coming along well!
One important change we should make is throwing when the index is out of bounds to match the native Array.prototype.with method.
| var bench = require( '@stdlib/bench' ); | ||
| var rand = require( '@stdlib/random/base/randu' ); | ||
| var pkg = require( '@stdlib/array/base/with/package.json' ).name; | ||
| var value; |
There was a problem hiding this comment.
Let's declare this variable inside the benchmark function at L34, as we don't need it in the outer scope.
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
make suggested changes Fixes stdlib-js#1328
|
@Planeshifter @kgryte I have done the suggested changes. |
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
|
@Pratik772846 Thanks for your contribution and good catch concerning |
|
@Pratik772846 Would you mind checking the box in the OP that you read the contributing guidelines? We need to ensure that box is checked for all external contributions. Thanks! |
updates value at a particular index
Fixes #1328
Resolves # Issue 1328
Description
This pull request:
Related Issues
This pull request:
Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers