CircleCI/Sider合同ミートアップ ツイートまとめ #circleci_sider
-
siderreview
- 645
- 0
- 0
- 0
-
- いいね！0
Koichiro, Sumi
@sumyapp
コードレビューでコードを引用するときはmasterではなくコミットへのリンクが必要 #codereview4 pic.twitter.com/3E9ualywuV
2018-09-27 19:29:38
拡大
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
伊藤さま。寿命が長いサービス、良いプロダクトほど、たくさんの開発者に読み継がれ、メンテナンスされていく。未来の読み手に引き継いでいくことも考えながら、コミットコメントやレビューを考えていきましょう。 #codereview4
2018-09-27 19:32:01
Koichiro, Sumi
@sumyapp
gfxさんの発表。コードレビューでは歴史的経緯の説明おじさんが必要(いる) #codereview4 pic.twitter.com/w5DMmiMVqk
2018-09-27 19:34:59
拡大
Koichiro, Sumi
@sumyapp
Sider社のQuerlyが良い、歴史的背景に良い github.com/soutaro/querly #codereview4
2018-09-27 19:36:58
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
gfxさま、タイトルは「歴史的経緯の説明 as code」 「コードレビューをしていると、どうしても歴史的経緯を説明するおじさんが出てきます」 この歴史的背景の説明には、だいたいパターンがある。 #codereview4
2018-09-27 19:37:18
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
本日のテーマはQuerly 。「歴史的経緯の説明 as code」というタイトルの通り、スタイルだけでなく、コードレビューに良く登場する『歴史的経緯を説明する』部分も、「おじさん」でなく自動解析にまかせてしまおうというもの。 #codereview4
2018-09-27 19:38:34
JN🐰
@allesgute_JN
gfxさまの講演です。 Sider社のQuerlyが良いとおっしゃいました。 #codereview4 pic.twitter.com/cgLa0n4Ofj
2018-09-27 19:39:22
拡大
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
Querlyの動きを紹介いただいてます。 「最初READMEを読んでも良くわからなかった」とのこと。 単純なパターンマッチではなく、Ruby ASTに対するマッチを行う。 メソッド呼び出しについて、引数が１つなのか配列なのかブロックなのか、といったパターンを検出できる。 #codereview4 #codereview4
2018-09-27 19:45:15
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
Querlyは誤検出もそこそこあるけれど、「このメソッドの使い方、ちょっとイケてないんじゃないですか？」みたいな、ふわっとした指摘ができる。 rubocopだとかなり直球で厳しい指摘が出る。（結果、だんだんルールがdisableにされちゃうことも...） #codereview4
2018-09-27 19:47:27
りんご🍏夜明けのリモートワーカー
@mstssk
「本当にメタプログラミングが必要か3回考えてください。」 #codereview4 pic.twitter.com/g4n99QnR0c
2018-09-27 19:47:51
拡大
Koichiro, Sumi
@sumyapp
QuerlyとSiderでSQLインジェクションされそうな、Raw SQLに注意喚起 #codereview4 pic.twitter.com/MWKqbGMZaY
2018-09-27 19:49:41
拡大
野球
@koalanosamba
Querlyは警告的なコードレビューを自動化できるツールで、メンバーの凡ミスを回避するのが◎なんだな #codereview4
2018-09-27 19:50:19
あきこ@しばらくしばらく趣味のつぶやき多め
@akiko_pusu
生SQLへの注意喚起。たとえばSQLに対する文字列、変数展開の箇所があったら検出したり。（「この変数展開大丈夫なの？SQLインジェクション大丈夫だよね？」くらいのツンツン的な指摘が出来るみたいです） #codereview4
2018-09-27 19:50:33
野球
@koalanosamba
パターンマッチを追加するとき、うまく動くかどうかは、どうやって試すんだろう? Badケースを書いてみて、querlyを実行するのかな? #codereview4
2018-09-27 19:51:41
星 永亮
@inase17000
@sumyapp さんのSider主催のミートアップに参加！！ Code Review Meetup #4 Code Reviewで集まろうsideci.connpass.com/event/98378/ #codereview4
2018-09-27 19:52:37
りんご🍏夜明けのリモートワーカー
@mstssk
Linterは一般的なルールを適用する。Querlyはプロジェクト固有のルール(歴史的経緯)を適用する。 github.com/soutaro/querly #codereview4
2018-09-27 19:53:13