chore: fix JavaScript lint errors (issue #10528)#10726
chore: fix JavaScript lint errors (issue #10528)#10726travisbreaks wants to merge 1 commit intostdlib-js:developfrom
Conversation
- Fix array-element-newline errors in stats/kstest/test/test.marsaglia.js by adding linebreaks between array elements - Remove unused eslint-disable directives for array-element-newline - Replace new Array() constructor with array literals and push in utils/pluck/benchmark/benchmark.js Co-authored-by: Egger <egger@horny-toad.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! |
|
Closing as this overlaps with existing work (Potential Duplicate label). Happy to reopen if maintainers would like me to continue. |
Description
Fixes the JavaScript lint errors reported in issue #10528.
Changes
lib/node_modules/@stdlib/stats/kstest/test/test.marsaglia.js:eslint-disable-line array-element-newlinedirectives on lines 46-47array-element-newlineerrors in theexpectedarray by placing each element on its own linelib/node_modules/@stdlib/utils/pluck/benchmark/benchmark.js:new Array( 100 )andnew Array( 5 )constructors with array literals andpush()to satisfy thestdlib/no-new-arraylint ruleRelated Issues
Resolves #10528
Co-authored-by: Egger egger@horny-toad.com
Checklist