はじめてのプルリクエスト

ハードル激低LT大会ッ! #02 で発表したスライド

Transcript

1. はじめてのプルリクエスト
   @cidermitaina
   2023.04.20 ハードル激低LT大会ッ！ #02
   

   View Slide

2. @cidermitaina
   - フロントエンドエンジニア　@SmartHR
   - UI作るのがすきです。
   - ひとみしりです。
   

   View Slide

3. 突然ですが、みなさん、
   

   View Slide

4. PRを作成してレビューをしてもらうとき、不安になったりし
   ませんか？🥺
   

   View Slide

5. 😢
   - たくさん指摘があったらどうしよう…
   - コメントがたくさんあると批判されてる気持ちになる…
   - 自分のコードに自信がない…
   

   View Slide

6. 💡 解決策
   

   View Slide

7. セルフレビュー😉
   

   View Slide

8. セルフレビューとは？
   レビュー依頼を出す前に、自分でレビューをすること。
   

   View Slide

9. セルフレビューで私がやっていること
   - 1行1行、頭の中で説明しながら読んでいく
   - なんでそのコードを書いたのか説明できない箇所は理解できていない証拠。
   - 自分のコードを客観的にレビューしてみる
   - 自分のコードではなく、他人のコードだと思ってレビューしてみる
   

   View Slide

10. やってみて感じたこと
    - 自分のコードに自信が持てるようになった
    - コードの質が上がった気がする
    - 変数・関数などの命名は適切か
    - 共通化できるところあるかどうか
    - 分離してすっきりできそうなところがあるか
    - レビューがスムーズになった
    - セルフレビューをすることで PRの精度が高くなって、レビュアーや自分の負担が減る
    

    View Slide

11. いいことたくさん👏
    

    View Slide

12. おわり👋
    

    View Slide