YukaMurata / modal

0 stars 1 forks source link

(追加要件)ESLintを利用してほしい。 #2

Closed liberta793 closed 7 years ago

liberta793 commented 7 years ago

「気になる点」で指摘した、スペースとタブの混同空行や末尾の不要なスペースなどに関してはESLintという構文解析ツールがあるのでそれを利用してください。

ESLintを利用する理由ですが、以下になります。

利用方法ですが、gulpや、コマンド、WebStormで実行したり手段は問いません。 ESLintを利用できるようにったら、このissueは閉じます。

YukaMurata commented 7 years ago

コマンドでファイルを指定して実行できるようになりました。 ただ、.eslintrcのenvにはほとんど追記していない状況です。

liberta793 commented 7 years ago

ありがとうございます。 .eslintrcを確認したいため、プッシュしていただけますか。

YukaMurata commented 7 years ago

プッシュしました。 ご確認よろしくお願いいたします。

liberta793 commented 7 years ago

ありがとうございます。 env箇所の入れ子のスペースが気になりました。

"env": {"browser": true,
"jquery": true},

理由や意図があるならそのままで問題ありませんが、 特になければ以下に修正していただけますか。

{
    "extends": ["eslint:recommended"],
    "plugins": [],
    "parserOptions": {},
    "env": {
        "browser": true,
        "jquery": true
    },
    "globals": {},
    "rules": {}
}

細かくて恐縮ですが、よろしくお願いいたします。

YukaMurata commented 7 years ago

ありがとうございます。 上記対応しましたので、ご確認よろしくお願いいたします。 1f6b4c8

liberta793 commented 7 years ago

確認できました!