bench: add missing method name in benchmark description#11155
bench: add missing method name in benchmark description#11155kgryte merged 1 commit intostdlib-js:developfrom
Conversation
The benchmark name format string was missing the :of: method name segment. Changed '%s:len=5' to '%s:of:len=5' to match the convention used by the first benchmark in the same file and other factory benchmarks. Closes stdlib-js#11151 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Hello! Thank you for your contribution to stdlib. We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:
This acknowledgment confirms that you've read the guidelines, which include:
We can't review or accept contributions without this acknowledgment. Thank you for your understanding and cooperation. We look forward to reviewing your contribution! |
Coverage Report
The above coverage report was generated for the changes in this PR. |
|
Hi, Just like in your other PR, we need the snippet suggested by the stdlib-bot to be added to the PR description Thanks! |
|
Added to the description. Thanks for the heads up. |
Summary
Adds missing
:of:method name to the benchmark format string in@stdlib/array/little-endian-factory/benchmark/benchmark.of.js.Changes
Line 55:
'%s:len=5'->'%s:of:len=5'The first benchmark in the file correctly uses
'%s:of'(line 36), but the second benchmark was missing the method name segment.Closes #11151
This contribution was developed with AI assistance (Claude Code).