Fix JavaScript lint errors

オープン 初心者向け
#13,209 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
70/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
静か
技術スタック
eslint, javascript
領域
tooling

調査の方向性

lib/node_modules/@stdlib/_tools/modules/pkg-deps/lib/to_array.js と lib/node_modules/@stdlib/blas/base/dsymv/test/test.ndarray.js から始め、報告された lint 箇所に重点を置いてください。workflow の詳細にある JavaScript lint target を実行し、報告された両方のエラーがなくなり、新たな lint 失敗がないことを確認してください。

索引モデルが issue の本文から書いたものです。

説明

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>".
主要言語
JavaScript
スター
6k
フォーク
1.3k
平均マージ
1日 1時間
マージ済み PR(30日)
559

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

stdlib-js/stdlib のほかの issue

stdlib-js/stdlib の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。