Upgrade to Pro — share decks privately, control downloads, hide ads and more …

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

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

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

ishitoko shiho

April 20, 2023
Tweet

Other Decks in Technology

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