Skip to content

chore: fix JavaScript lint errors (issue #13209)#13656

Open
mantrapatel05 wants to merge 3 commits into
stdlib-js:developfrom
mantrapatel05:fix-lint-13209
Open

chore: fix JavaScript lint errors (issue #13209)#13656
mantrapatel05 wants to merge 3 commits into
stdlib-js:developfrom
mantrapatel05:fix-lint-13209

Conversation

@mantrapatel05

Copy link
Copy Markdown
Contributor

Description

Fixed two lint errors:

  • Replaced new Array(n) with an array literal + .push() in to_array.js (stdlib/no-new-array)
  • Removed a stray blank line between two require statements in dsymv's test.ndarray.js (stdlib/no-empty-lines-between-requires)

Note: I noticed #13210 already addresses this same issue, but it appears to have stalled — the contributing-guidelines check has been failing for a month with no maintainer review. Opening this as an alternative in case that one needs to be superseded or closed.

Related Issues

resolves #13209

@mantrapatel05
mantrapatel05 requested a review from a team July 25, 2026 18:49
@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 25, 2026
@stdlib-bot

Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
blas/base/dsymv $\\color{green}470/470$
$\\color{green}+100.00\\%$
$\\color{green}75/75$
$\\color{green}+100.00\\%$
$\\color{green}2/2$
$\\color{green}+100.00\\%$
$\\color{green}470/470$
$\\color{green}+100.00\\%$

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

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. Needs Review A pull request which needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix JavaScript lint errors

2 participants