Repository navigation
Fix JavaScript lint errors #15176
Copy link
Copy link
Closed
Labels
Good First IssueA good first issue for new contributors!A good first issue for new contributors!
Description
JavaScript Linting Failures
Linting failures were detected in the automated JavaScript lint workflow run.
Workflow Details
- Run: https://github.com/stdlib-js/stdlib/actions/runs/34660299012
- Type: JavaScript Linting
- Date: 2026-09-12 00:15:06 UTC
Error Details
make[1]: Entering directory '/home/runner/work/stdlib/stdlib'
Linting file: lib/node_modules/@stdlib/ndarray/base/assign/benchmark/benchmark.6d_blocked_rowmajor.js
Linting file: lib/node_modules/@stdlib/array/float16/benchmark/polyfill/benchmark.find_last_index.length.js
/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/array/float16/benchmark/polyfill/benchmark.find_last_index.length.js
19:1 error Unused eslint-disable directive (no problems were reported from 'stdlib/jsdoc-typedef-typos')
✖ 1 problem (1 error, 0 warnings)
1 error and 0 warnings potentially fixable with the `--fix` option.
Linting failed.
Linting file: lib/node_modules/@stdlib/array/complex64/benchmark/benchmark.subarray.length.js
Linting file: lib/node_modules/@stdlib/stats/base/dists/hypergeometric/mean/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/math/base/special/atan2/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/blas/base/diagonal-type-resolve-enum/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/blas/ext/base/cdiff/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/number/float64/base/assert/is-same-value/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/stats/base/dists/gamma/logcdf/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/number/float64/base/to-words/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/fft/base/fftpack/generic/sinti/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/ndarray/base/reinterpret-complex64/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/blas/base/drotm/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/stats/base/dists/chisquare/stdev/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/stats/strided/dsmeanwd/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/blas/ext/base/drssbl/benchmark/benchmark.native.js
Linting file: lib/node_modules/@stdlib/blas/base/sswap/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/array/fixed-endian-float32/benchmark/benchmark.to_string.length.js
Linting file: lib/node_modules/@stdlib/stats/strided/dsnanmeanors/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/blas/ext/base/gindex-of-almost-same-value/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/ndarray/base/clamp-index/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/math/iter/special/tan/benchmark/benchmark.js
Linting file: lib/node_modules/@stdlib/ndarray/base/assert/is-integer-data-type/benchmark/benchmark.js
make[1]: *** [/home/runner/work/stdlib/stdlib/tools/make/lib/lint/javascript/eslint.mk:261: eslint-files] Error 1
make[1]: Leaving directory '/home/runner/work/stdlib/stdlib'
make: *** [/home/runner/work/stdlib/stdlib/tools/make/lib/lint/javascript/Makefile:158: lint-javascript-files] Error 2
Pull Request Instructions
- Please use the following PR title format:
"chore: fix JavaScript lint errors (issue #<ISSUE_NUMBER>)". - Reference this issue in the "Related Issues" section of the PR body as "resolves #<ISSUE_NUMBER>".
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Good First IssueA good first issue for new contributors!A good first issue for new contributors!
Activity
stdlib-bot commentedon Sep 12, 2026
👋 Important: PLEASE READ 👋
This issue has been labeled as a good first issue and is available for anyone to work on.
If this is your first time contributing to an open source project, some aspects of the development process may seem unusual, arcane, or some combination of both.
Before working on this issue and opening a pull request, please read the project's contributing guidelines. These guidelines and the associated development guide provide important information, including links to stdlib's Code of Conduct, license policy, and steps for setting up your local development environment.
To reiterate, we strongly encourage you to refer to our contributing guides before beginning work on this issue. Failure to follow our guidelines significantly decreases the likelihood that you'll successfully contribute to stdlib and may result in automatic closure of a pull request without review.
Setting up your local development environment is a critical first step, as doing so ensures that automated development processes for linting, license verification, and unit testing can run prior to authoring commits and pushing changes. If you would prefer to avoid manual setup, we provide pre-configured development containers for use locally or in GitHub Codespaces.
We place a high value on consistency throughout the stdlib codebase. We encourage you to closely examine other packages in stdlib and attempt to emulate the practices and conventions found therein.
In short, the more effort you put in to ensure that your contribution looks and feels like stdlib—including variables names, bracket spacing, line breaks, etc—the more likely that your contribution will be reviewed and ultimately accepted. We encourage you to closely study the codebase before beginning work on this issue.
✨ Thank you again for your interest in stdlib, and we look forward to reviewing your future contributions. ✨
akkupratap323 commentedon Sep 12, 2026
I addressed the reported unused lint directive in #15177.
Abhist17 commentedon Sep 17, 2026
I looked into this. lib/node_modules/@stdlib/array/float16/benchmark/polyfill/benchmark.find_last_index.length.js:19 carries a file-level /* eslint-disable stdlib/jsdoc-typedef-typos */ directive, but the file has no @typedef or @type JSDoc annotations at all for that rule to check, so ESLint's reportUnusedDisableDirectives flags it as unused. Every sibling benchmark file in the same lib/node_modules/@stdlib/array/float16/benchmark/polyfill/ directory (copy_within, at, fill, …) has no such directive, so this one is a stray left over from an earlier refactor, not a live suppression. I'd fix it by delete the unused eslint-disable comment on line 19. Happy to take this if it's unassigned.
Abhist17 commentedon Sep 17, 2026
Opened #15297 for this (linked with Fixes #15176). Review comments welcome — I'll turn them around the same day.
chore: fix JavaScript lint errors