Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use eslint #10

Merged
merged 3 commits into from
Jan 21, 2020
Merged

Use eslint #10

merged 3 commits into from
Jan 21, 2020

Conversation

egs33
Copy link
Member

@egs33 egs33 commented Jan 7, 2020

ESLintを導入し, ソースを整形/チェックする。
eslintのルールはextendしたものを基本的にデフォルトのまま使用しています。

Diffのサイズが大きいのでESLintの導入のみのPRにしています

@egs33 egs33 marked this pull request as ready for review January 21, 2020 01:13
@egs33 egs33 requested a review from saitouena January 21, 2020 01:23
Copy link
Contributor

@saitouena saitouena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@egs33
私はeslintをよく知らないので設定の良し悪しはわかりませんが、linterなのでさっさとマージしてしまいます。ちょっと今仕事が溜まっているので、ちゃんと見る余裕がないです。申し訳ないです。

@saitouena saitouena merged commit 79e4a6a into master Jan 21, 2020
@saitouena saitouena deleted the feature/introduce_eslint branch January 21, 2020 02:04
@saitouena
Copy link
Contributor

@egs33
有識者によるアドバイスを受けたので一応共有しておきます。

air-bnbとかstandard-jsとか定評あるルールセット導入すると幸せになれるかもしれない
あと、 @typescript-eslint/no-unused-vars 使うなら通常の no-unused-vars はoffにしよう。(意図せぬ挙動をしてしまう)

試してみて良さそうだったらまたPRにしてもらえると嬉しいです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants