Openlibrary: JavaScriptをeslintでリントしましょう！

作成日 2018年10月17日 · 5コメント · ソース: internetarchive/openlibrary

Stylelintは、さまざまなCSSの問題からの保護を提供します。 JSについても同じようにしましょう。
eslintは最も成熟しており、必要と思われる場合に利用できるルールのリストを提供

npm install eslint --save-dev

package.jsonに、新しいコマンドを追加します

"lint:js": "eslint openlibrary/plugins/openlibrary/js/"

.eslintrc.jsonを追加します

{
    "root": true,
    "env": {},
    "globals": {
    },
    "rules": {
        "no-extra-semi": "warn"
    }
}

とりあえず警告するようにエラーを設定します。構成がマージされたらすぐに、誰かと調整して修正を実行し、マージの競合を回避する必要があります。

[] npm run lint：jsは、「no-extra-semi」ルールに関連する警告を表示します。
[] "npm run lint：js "は、 "npm runtest"を実行すると発生します
[]この変更の一部としてエラーを修正しないでください！

Good First Issue Linting JavaScript 1 hacktoberfest

ソース

jdlrobson

全てのコメント5件

こんにちは。この問題を解決するために私を割り当てることができますか？

0x15F9 2018年10月18日

質問： eslintのプリセットの1つを使用しないのはなぜですか？ https://standardjs.comやhttps://github.com/airbnb/javascriptのように

tamtamchik 2018年10月20日

長期的にはやりたいのですが、JavaScriptの状態が悪いため、このようなプリセットを適用すると、多くの混乱（コードレビュー、リベースの競合）と手作業（ほとんどのプリセットは自動修正できません）になります。ルールを手動で追加すると、まとまりのある教育的な方法で少しずつ攻撃するのに役立つと思います（コードの問題について少しずつ学びます）。

リンティングを使用して、最も一般的で差し迫った問題（コードでエラーを引き起こす可能性のあるリント）を解決し、後で（選択した）プリセットを適用できると考えました。

これは意味がありますか？

jdlrobson 2018年10月20日

👍1

確かに、そうです。コードの形が非常に悪い場合があります。

このPRがマージされるとき、私はリンティングに関するいくつかの問題を修正することができます。ただemと名付けてください。
たとえば、 no-extra-semiで始まります;）

tamtamchik 2018年10月20日

👍1

@tamtamchik後で問題を作成できますが、「エラーの可能性」の下にあり、自動修正可能なものはすべて公正なゲームです。

@mekarpelesと調整して、

ぶら下がっている果物を取り除きたい！

jdlrobson 2018年10月21日

🎉1

このページは役に立ちましたか？

0 / 5 - 0 評価

Openlibrary: JavaScriptをeslintでリントしましょう！

全てのコメント5件

関連する問題