sfc-rg / rg-portal

Internal Website for Keio Univ. Technology joint research groups
https://portal.sfc.wide.ad.jp/
12 stars 5 forks source link

HoundCIの設定ファイルを追加 #10

Closed demmys closed 9 years ago

demmys commented 9 years ago

某所で使用しているrubocop.ymlを追加してみたけど、どうだろう。 ちょっと変えないとよくなさそう。

demmys commented 9 years ago

あっ、いつもの癖でコミットメッセージ日本語にしてた。まあいいよね。 @miyukki どうだろう。

miyukki commented 9 years ago

これすごい長いから確認に時間掛かりそう

miyukki commented 9 years ago

もしくはマージして不都合でたらそれに合わせてもあり… というより検証用のリポジトリ用意してGoodとBadパターンが正しく動くか検証するのがいいと思った

demmys commented 9 years ago

@miyukki だいたい自分がやってるプロジェクトではとりあえずマージして不都合が出たら返るパターンだな。 ABCサイズの制限だけ厳しすぎる気がしたからゆるめた。

miyukki commented 9 years ago

いちおうマージ前に別のリポジトリでテストしてみる

demmys commented 9 years ago

@miyukki お、すばらしい。たのんだ

miyukki commented 9 years ago

ちょっと修正してあげる必要がありそう https://github.com/miyukki/hound-ci-test/pull/1/files

demmys commented 9 years ago

@miyukki hound-ci-testのやつ、ぜんぶの良くないところにコメントが付いてるわけじゃないみたいだ。 PRの最後のところには"114 violations found."って書いてあるけど、コメントは16個しかついてない... とりあえずローカルでrubocopを動かして確認してみたら幾つか良くないのがあったから、修正してみる。

demmys commented 9 years ago

@miyukki https://github.com/demmys/hound-ci-test/pull/5 こんなふうにエラーが出るようにしてみた。 badケースなのに吠えてくれない、みたいなヤツについてはやりだすとすごい量だから、追々追加していく形でいいかなと。

demmys commented 9 years ago

@miyukki SCSSの設定ファイルも追加した。

miyukki commented 9 years ago

とりあえず追加しちゃいましょうか。

miyukki commented 9 years ago

LGTM

demmys commented 9 years ago

しましょう。