Skip to content

chore: fix JavaScript lint errors (issue #13114) - #13630

Closed
mantrapatel05 wants to merge 1 commit into
stdlib-js:developfrom
mantrapatel05:fix/javascript-lint-13587
Closed

mantrapatel05 wants to merge 1 commit into
stdlib-js:developfrom
mantrapatel05:fix/javascript-lint-13587

Conversation

@mantrapatel05

Copy link
Copy Markdown
Contributor

Description

Replaced new Array(n) with array literals + .push() in benchmark.js to fix the stdlib/no-new-array errors (12 of them).

Also checked the doctest error mentioned in the issue (read-file-list/examples/index.js) but couldn't reproduce it on current develop — the output already matches the comments. Might've been fixed already, happy to look again if needed.

Related Issues

resolves #13114

  • Read, understood, and followed the [contributing guidelines]

@mantrapatel05
mantrapatel05 requested a review from a team July 23, 2026 15:11
@stdlib-bot stdlib-bot added Needs Review A pull request which needs code review. Good First PR A pull request resolving a Good First Issue. labels Jul 23, 2026
@stdlib-bot

stdlib-bot commented Jul 23, 2026 •

Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
stats/chi2gof $\\color{red}955/983$
$\\color{green}+0.00\\%$
$\\color{red}103/114$
$\\color{green}+0.00\\%$
$\\color{red}13/15$
$\\color{green}+0.00\\%$
$\\color{red}955/983$
$\\color{green}+0.00\\%$

The above coverage report was generated for the changes in this PR.

@mantrapatel05
mantrapatel05 force-pushed the fix/javascript-lint-13587 branch from 5a6367b to a102a2b Compare July 23, 2026 15:59
@stdlib-bot stdlib-bot added the Statistics Issue or pull request related to statistical functionality. label Jul 23, 2026
@mantrapatel05
mantrapatel05 deleted the fix/javascript-lint-13587 branch July 23, 2026 16:04
@stdlib-bot stdlib-bot removed the Needs Review A pull request which needs code review. label Jul 23, 2026
@mantrapatel05
mantrapatel05 restored the fix/javascript-lint-13587 branch July 23, 2026 16:05
@mantrapatel05
mantrapatel05 deleted the fix/javascript-lint-13587 branch July 23, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Good First PR A pull request resolving a Good First Issue. Statistics Issue or pull request related to statistical functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix JavaScript lint errors

2 participants