コードレビュー時のラベルの付け方

Transcript

1. コードレビュー時の ラベルの付け方 週明けからアルカナの人になる 今山 豪貴
2. ✨自己紹介✨ なまえ: 今山豪貴 せいべつ: おとこ しょくぎょう: ぺちぱーLV13（年目） クエストりれき: 光の戦士 → 塾教務 → 船橋のSES会社 → 九段下のSES会社（オーガニックママ →写真SNS→特許 検索→スマートパス→ソシャゲ→ownCloud→コールセンターのシステム →家電メーカーの会員サー ビス） → アンテク（家電メry/アヒル屋/おもちゃ屋/百貨店/質屋→マッチング♡サイト→香り AI） → スタジオアルカナ
3. 📢どうしてこの話をしようと思ったの？📢 1. 3/8にまぐろさんがslackにこんなことを投稿した → 2. 内容を読んでみて、 「あ、これアンテク内でもやってましたわ」 3. ということであればどんなふうにやってたの？ と、まぐろさんに言われたので、これネタ🍣にしよ って思いました
4. こんなことなかったですか？
5. 💀レビューコメントにまつわる不幸💀 • シンプルに質問しただけのつもりが、反語表現的に取られてあまり本質的で ない修正を入れられてしまった😢 • シンプルに質問しただけなのに、煽りだと受け取られてレビュイーの機嫌を 損ねてしまった💢 • 絶対に直してほしいのに、遠回しな言い回しをしたせいか無視された😨 • 別に後で直してくれたらいいなーって思ってただけなのに一生懸命直されて納期を ぶっ千切られてしまった✂
6. ラベルつけてみようぜ
7. ✌アンテクではこんな感じにしてました✌
8. 📔ラベルの区分（きほん）📔 • must ◦ 絶対直してね！　マジで！　動かなくなるから！！ ◦ 全然要求仕様と違うよ！　お客さんに激怒されるよ！！ ◦ LinterのところでCI止まったよ！　マージができないよ！！ ◦ コンフリクトしたよ！！　マージがで (ry • ask(Q) ◦ あれ、こんな仕様だったっけ？ ◦ この変数名、どういう意味〜？ • imo ◦ まあこれでもええけど、ワイならこうするな〜 • nits ◦ めっちゃ細かいんやけど、単数形より複数形のほうがええな〜
9. 📔ラベルの区分（おうよう）📔 • suggest ◦ 修正コードを提示する ◦ 今山は割と多用する • should ◦ mustとimoのあいだくらい ◦ こんな感じに直したほうがいいんじゃない〜？　強制はしないけど〜 • boyaki ◦ おっちゃんの独り言なんやけどな、もうちょっとね、こう …… ◦ 無視してくれてもええんやで ……
10. 良きプルリク生活を ご清聴ありがとうございました