search

syunto07ka / jewelry_box

2 stars 0 forks source link

ts及びtsxファイルに対してコードフォーマットをかける #52

Closed piro0919 closed 4 years ago

piro0919 commented 4 years ago

Is your feature request related to a problem? Please describe. 規約を守れていないコードを書いた場合でもcommitを上げることが可能な状態となっている

Describe the solution you'd like huskyにコードフォーマットをかませる

Additional context フォーマッターの選定はおまかせします 👍

syunto07ka commented 4 years ago

これはコミットメッセージの話ではなく。。？

piro0919 commented 4 years ago

@syunto07ka

コミットメッセージは関係ないですよ。 👍

単純にprettierとかeslintあたりの話ですね。

syunto07ka commented 4 years ago

issueのクローズ条件は

ですかね

piro0919 commented 4 years ago

@syunto07ka

huskyにgitのcommit時にコードフォーマットをする設定をする

こっちは認識あってます 👍

Prettierのインストール

こっちは prettier じゃなくても問題ないです、もちろん prettier でも良いですが この前 eslint で完結させたいみたいなこと言われてませんでしたっけ？

syunto07ka commented 4 years ago

あまり知識ないときに言ってたことなので、もうちょっと調べてどうするか決めようかなと。 どこかでちらっとESLintとPrettierは使い分けたほうがよいと書いてあったのを見た気がした

どこかでちらっとESLintとPrettierは使い分けたほうがよいと書いてあったのを見た気がした

piroさん以外で、ですね

syunto07ka commented 4 years ago

https://prettier.io/docs/en/integrating-with-linters.html#eslint

eslint-config-prettier is a config that disables rules that conflict with Prettier. Add it to your devDependencies, then extend from it within your .eslintrc configuration. Make sure to put it last in the extends array, so it gets the chance to override other configs

eslint-config-prettier はprettierと競合するeslintのルールを消す

eslint-plugin-prettier is a plugin that adds a rule that formats content using Prettier. Add it to your devDependencies, then enable the plugin and rule.

prettierのルールをeslintで使うためのプラグイン