yasasii-team / Stepippo-iOS

初めての一歩を応援するアプリのiOS版
https://github.com/yasasii-team/Stepippo-iOS.git
21 stars 5 forks source link

SwiftLintを導入し、Run Scriptにスクリプトを記述(設定)した。 #121

Closed eyener3 closed 5 years ago

eyener3 commented 5 years ago

fixes #114

(#の後にcloseしたいissueの番号を記述してください)

Summary(要約)

SwiftLintを導入し、Run Scriptにスクリプトを記述(設定)した。

115 差分に余計なものが入っていたため、cherrypickを行ないプルリクし直しました。

Other Information(他の情報)

質問があります💡m(._.)m SwiftLintのルール選定はプルリク時に、少しずつやっていく感じに落ち着いたのでしたか?^-^ 最後の結論がどうだったかなと、記憶が曖昧になってしまいました。。><

Tested(テストしたこと)

Must Reviewer(必須レビュアー)

riscait commented 5 years ago

確か、そのまま適用するとライブラリ等にもルール適用されてしまい大変だから、 ①導入するならまずはデフォルトルールを無効にするルールを設定しないといけなかったと思います!

②そしてまだルールも定まっていないのでそれは今後の実装PRで議論しながら定めていく (Swiftコーディングルールを作るissueがあるのでそこにまとめていきましょう!)

だったかと思います。

なので手が空いてる人や詳しい人が①をやりますが、優先度低い(アプリのリリースには必要ない)ので、他にやること(画面実装など)あればLintには手をつけなくてもよい、という認識です🙆‍♂️

Sent with GitHawk

Koki-Nakamura commented 5 years ago

村松さんと同じ認識です。 基本的にはmasterはエラーが無い状態をキープする必要がありますので、エラーが出ている状態でPR出すのはNGです。 ①の対応私がやっても大丈夫ですが、せっかく導入いただいたのであいなーさんにもマスターしてもらいたいと思ってます。 なので1度やってみて頂いてわからなければ画面共有でもしながら一緒にやってみるのはアリだと思っています。 しかし優先順位は低いと思いますので別のタスクからやっていただいても構いません。

eyener3 commented 5 years ago

龍之介さん、Kokiさん、 ありがとうございます!m(._.)m

スクリプトの記述はなしにしました^-^🙏 また余裕がある時に挑戦してみようと思います