Fix JavaScript lint errors

Open Beginner friendly
#13,209 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
70/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
eslint, javascript
Domain
tooling

Research direction

Start with lib/node_modules/@stdlib/_tools/modules/pkg-deps/lib/to_array.js and lib/node_modules/@stdlib/blas/base/dsymv/test/test.ndarray.js, focusing on the reported lint locations. Run the JavaScript lint target from the workflow details and confirm both reported errors are gone with no new lint failures.

Written by the indexing model from the issue text.

Description

Good First Issue

JavaScript Linting Failures

Linting failures were detected in the automated JavaScript lint workflow run.

Workflow Details
Error Details
make[1]: Entering directory '/home/runner/work/stdlib/stdlib'

Linting file: lib/node_modules/@stdlib/stats/strided/stdev/lib/stdev.js

Linting file: lib/node_modules/@stdlib/blas/ext/base/snancount/lib/main.js

Linting file: lib/node_modules/@stdlib/complex/float32/base/sub/lib/index.js

Linting file: lib/node_modules/@stdlib/stats/strided/dztest2/lib/main.js

Linting file: lib/node_modules/@stdlib/strided/base/unary-addon-dispatch/lib/main.js

Linting file: lib/node_modules/@stdlib/stats/base/dists/rayleigh/logpdf/lib/index.js

Linting file: lib/node_modules/@stdlib/stats/array/max-by/lib/index.js

Linting file: lib/node_modules/@stdlib/random/array/f/lib/factory.js

Linting file: lib/node_modules/@stdlib/math/base/special/gamma-lanczos-sum-expg-scaled/lib/native.js

Linting file: lib/node_modules/@stdlib/blas/ext/base/saxpby/lib/index.js

Linting file: lib/node_modules/@stdlib/array/float32/lib/polyfill.js

Linting file: lib/node_modules/@stdlib/ndarray/dispatch-by/lib/main.js

Linting file: lib/node_modules/@stdlib/strided/base/smap/lib/index.js

Linting file: lib/node_modules/@stdlib/ndarray/base/nullary-strided1d-dispatch/lib/main.js

Linting file: lib/node_modules/@stdlib/assert/napi/is-dataview/lib/native.js

Linting file: lib/node_modules/@stdlib/array/base/rekey/lib/main.js

Linting file: lib/node_modules/@stdlib/_tools/modules/pkg-deps/lib/to_array.js

/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/_tools/modules/pkg-deps/lib/to_array.js
40:8  error  Using the `new Array()` constructor is not allowed; use an array literal with push instead  stdlib/no-new-array

✖ 1 problem (1 error, 0 warnings)

Linting failed.

Linting file: lib/node_modules/@stdlib/random/shuffle/lib/index.js

Linting file: lib/node_modules/@stdlib/random/streams/normal/lib/debug.js

Linting file: lib/node_modules/@stdlib/math/strided/special/strunc/lib/main.js

Linting file: lib/node_modules/@stdlib/array/base/zeros4d/lib/main.js

Linting file: lib/node_modules/@stdlib/string/base/format-interpolate/lib/is_number.js

Linting file: lib/node_modules/@stdlib/plot/components/svg/axis/lib/methods/render.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
make[1]: Entering directory '/home/runner/work/stdlib/stdlib'

Linting file: lib/node_modules/@stdlib/math/base/utils/relative-difference/test/test.js

Linting file: lib/node_modules/@stdlib/ndarray/concat/test/test.main.js

Linting file: lib/node_modules/@stdlib/stats/strided/sdsmean/test/test.sdsmean.js

Linting file: lib/node_modules/@stdlib/datasets/harrison-boston-house-prices-corrected/test/test.js

Linting file: lib/node_modules/@stdlib/array/base/flatten-by/test/test.assign.js

Linting file: lib/node_modules/@stdlib/complex/float32/base/add/test/test.assign.js

Linting file: lib/node_modules/@stdlib/_tools/github/following/test/test.js

Linting file: lib/node_modules/@stdlib/blas/ext/base/cwapx/test/test.cwapx.js

Linting file: lib/node_modules/@stdlib/datasets/suthaharan-multi-hop-sensor-network/test/test.cli.js

Linting file: lib/node_modules/@stdlib/blas/base/dsymv/test/test.ndarray.js

/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/base/dsymv/test/test.ndarray.js
40:1  error  Unexpected empty line between require statements  stdlib/no-empty-lines-between-requires

✖ 1 problem (1 error, 0 warnings)
1 error and 0 warnings potentially fixable with the `--fix` option.

Linting failed.
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>".
Dominant language
JavaScript
Stars
6k
Forks
1.3k
Avg merge
1d 1h
Merged PRs (30d)
559

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from stdlib-js/stdlib

All issues in stdlib-js/stdlib

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.