[tamarubykaigi01]プロになるためのレビューのススメ

しげる @a4geru

use.get_email => bad user.email => better find_by_id(user) => bad find_by_id(user_id) => better 確かに... #tamarubykaigi01

hirapi🍔 @chmv71

自分よりレベルの高い人のコード、レビューするのめちゃくちゃ楽しいですよね！！ なんなら仕事で一番好き #tamarubykaigi01

neko+ @neko314_

レビューしたりされたりが成長につながるわかる #tamarubykaigi01

ぷぽ @pupupopo88

わかる...自分よりレベルの高い人のレビューしたい... #tamarubykaigi01

森塚　真年@エンペイ @sanfrecce_osaka

レビューで技を盗める #tamarubykaigi01

sue445 @sue445

Rails仕草、レビューで指摘されないと気づかないので機械で検出できるようにしたい（ある程度はrubocopできそうだけど） #tamarubykaigi01

kappaZZ @kpinput

find_eachを知らな買ったｗｗｗ これだけでも勉強になっている #tamarubykaigi01

安尾友佑 | ラクスル @yusuke_yasuo

3. プロの技を盗める。これもめちゃ分かる。 #tamarubykaigi01

森塚　真年@エンペイ @sanfrecce_osaka

コードを通してコミュニケーションが増える #tamarubykaigi01

tma2 @tma2k

二日酔いやばい... #tamarubykaigi01

安尾友佑 | ラクスル @yusuke_yasuo

4. コードから始まるコミュニケーション #tamarubykaigi01

expa / Shu Oogawara @expajp

「これすごいな」と思った箇所をレビューしたときはちゃんとその行に 👍 って入れてる #tamarubykaigi01

ぷぽ @pupupopo88

OSSのコード見るのもいいんだけど、最初どこ見たらいいかわからないし、物によってはあまりよくないのもあるのでその勘所掴むのが難しいんですよね #tamarubykaigi01

しげる @a4geru

「仕様を把握する」その観点でレビューするの難しいけど、確かに非常時の対応とかしやすくなると思う。 #tamarubykaigi01

tma2 @tma2k

プロの技を盗めるはとてもあり、 一部聖書としてベテランのコードを新しい人に見せたら宗教っぽくなった。。。 #tamarubykaigi01

Yukako @__yucatty

レビューすると他の人のタスクも理解できる #tamarubykaigi01

安尾友佑 | ラクスル @yusuke_yasuo

6. 自分ならどう実装するか比較する #tamarubykaigi01

森塚　真年@エンペイ @sanfrecce_osaka

レビューしながら自分ならどう実装するかを考えてみる #tamarubykaigi01

hirapi🍔 @chmv71

開発コンテキストにあわせた実装、個人開発では得にくくてチーム開発で学べることの代表だと思う #tamarubykaigi01

のだみ @n0m1n0z

自分ならどう実装するか比較、わかる #tamarubykaigi01

tma2 @tma2k

自分ならどう書くかとの差分を考える #tamarubykaigi01

うなすけ @yu_suke1994

適切な単位でpull reqを作る、身に覚えがあります #tamarubykaigi01

hirapi🍔 @chmv71

1つのプルリクが巨大になるの、よくやってしまう…… #tamarubykaigi01

しげる @a4geru

PRの作り方わかる... 差分は少なくだなー 5ファイルくらいからつらい...って思っちゃう(ちゃんとみるけど #tamarubykaigi01

森塚　真年@エンペイ @sanfrecce_osaka

最初PRの差分がとんでもないことになるの初心者あるあるすぎる😇 #tamarubykaigi01