Less.js: [メンテナンスタスク] -ESLint設定を確認し、リンティングの問題を修正します

作成日 2018年06月21日  ·  6コメント  ·  ソース: less/less.js

しばらく前に、プロジェクトをJSHintからESLintに切り替えました(後者の方が保守がよく、より多くの機能を提供しています)。 ただし、潜在的な問題に対処する時間がなかったため、多くの設定を警告として設定しました。 また、コードをより保守しやすくする設定のリンティングを微調整する時間もありませんでした。

したがって、タスクは基本的にESLint警告を調べ、安全に対処できるものを確認することです。 (たとえば、 ==同等性警告の多くは、そのチェックを通過するすべての値をテストしないと、 === 「修正可能」ではない場合があります。)次の警告も設定すると便利です。 JSDocドキュメントの実施など、将来/継続的に取り組むべきこと。

good first issue medium priority up-for-grabs

全てのコメント6件

こんにちは、これを手に入れたいです!

@umuur頑張って!

@ matthew-dean現在のeslint構成を維持しますか?
TypeScriptをパーサーとして使用し、プラグインが必要かどうかはわかりません。

@umuur

現在のeslint構成を維持しますか?

場合によります。 コードのフォーマットを変更しない変更を加えたい場合は、それで問題ありません。 しかし、警告に関しては、IMOはすべて有効な警告であり、それらに対処するか、無効化されたeslintと(最も重要なことですが)そこで無効な理由についてコードコメントを残します。

現在のeslint構成を維持しますか?

コードベースはまだTypeScriptを使用していませんが、私の経験では、Babelよりも優れたトランスパイルを実行します。 ESLintに関しては....🤔そうです、TypeScriptにコードが含まれるまで、技術的には必要ありません。 私はおそらくコードベースをTSに変換することを見越してそれをしましたが、今はそれが起こるかどうか/いつ起こるかわかりません。

@umuur

ちなみに、paramsに適切なタイプのJSDocコメントを追加できるところならどこにでも追加してください!

@ matthew-ディーン詳細なコメントありがとうございます! アップデートについてお知らせします。

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

bassjobsen picture bassjobsen  ·  6コメント

Oskariok picture Oskariok  ·  6コメント

moshemo picture moshemo  ·  7コメント

chricken picture chricken  ·  6コメント

pknepper picture pknepper  ·  3コメント