Skip to content

chore: address JavaScript lint error#12221

Open
Aj242005 wants to merge 3 commits into
stdlib-js:developfrom
Aj242005:fix/issue-12208
Open

chore: address JavaScript lint error#12221
Aj242005 wants to merge 3 commits into
stdlib-js:developfrom
Aj242005:fix/issue-12208

Conversation

@Aj242005
Copy link
Copy Markdown
Contributor

@Aj242005 Aj242005 commented May 20, 2026

Resolves #12208

Description

What is the purpose of this pull request?
This pull request:

  • The Javascript lint workflow was flagging an error at 2 places
  • 1st one at lib/node_modules/@stdlib/plot/components/svg/defs/lib/main.js at 79th line , were previously there was a preallocated size array using
    args = new Array ( arguments.length + 1 ); args[0] = 'render' ; for ( i = 0; i<arguments.length;i++ ){ args[i+1] = arguments[i] } -> the lint workflow was flagging it because of presize allocation of array, i changed it to
    args = [ 'render' ]; for( i = 0; i<arguments.length; i++ ){ args.push( arguments[ i ] ); }
  • 2nd one at lib/node_modules/@stdlib/blas/base/ssyr/test/test.ndarray.js where there was an unexpected blank line at line 43, so the lint workflow was flagging it too --> removed it.

Related Issues

Does this pull request have any related issues?
This pull request has the following related issues:

Questions

Any questions for reviewers of this pull request?
No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.
No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

AI Assistance

When authoring the changes proposed in this PR, did you use any kind of AI assistance?

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

If you answered "yes" to using AI assistance, please provide a short disclosure indicating how you used AI assistance. This helps reviewers determine how much scrutiny to apply when reviewing your contribution. Example disclosures: "This PR was written primarily by Claude Code." or "I consulted ChatGPT to understand the codebase, but the proposed changes were fully authored manually by myself.".


@stdlib-js/reviewers

Aj242005 added 2 commits May 21, 2026 00:47
…flow issue

Removed a blank line (previously at line 43) which was causing a lint workflow issue

Signed-off-by: Akshit Jain <akshitjain6661@gmail.com>
…ne 79th )

Signed-off-by: Akshit Jain <akshitjain6661@gmail.com>
@Aj242005 Aj242005 requested a review from a team May 20, 2026 19:46
@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 May 20, 2026
@stdlib-bot
Copy link
Copy Markdown
Contributor

stdlib-bot commented May 20, 2026

Coverage Report

Package Statements Branches Functions Lines
blas/base/ssyr $\color{green}616/616$
$\color{green}+0.00%$
$\color{green}82/82$
$\color{green}+0.00%$
$\color{green}5/5$
$\color{green}+0.00%$
$\color{green}616/616$
$\color{green}+0.00%$
plot/components/svg/defs $\color{green}215/215$
$\color{green}+0.00%$
$\color{green}9/9$
$\color{green}+0.00%$
$\color{green}4/4$
$\color{green}+0.00%$
$\color{green}215/215$
$\color{green}+0.00%$

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

@Aj242005 Aj242005 changed the title Fix/issue 12208 chore: address JavaScript lint error #12208 May 20, 2026
@Aj242005 Aj242005 changed the title chore: address JavaScript lint error #12208 chore: address JavaScript lint error May 20, 2026
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