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

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

Avatar for ishitoko shiho ishitoko shiho
April 20, 2023
Technology
0
1.7k

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

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

Avatar for ishitoko shiho

ishitoko shiho

April 20, 2023
Tweet

More Decks by ishitoko shiho

See All by ishitoko shiho
はじめて tsgo について興味を持ったときに私の語ること
Avatar for ishitoko shiho cidermitaina
0
1k

Other Decks in Technology

See All in Technology
越境する組織づくり ─ 多様性を前提にしたチームビルディングとリードの実践知
Avatar for hrpnx kido_engineer
2
190
組織全体で実現する標準監視設計
Avatar for Yuto Obayashi yuobayashi
3
480
最強のAIエージェントを諦めたら品質が上がった話 / how quality improved after giving up on the strongest AI agent
Avatar for kt2 kt2mikan
0
160
Abuse report だけじゃない。AWS から緊急連絡が来る状況とは？昨今の攻撃や被害の事例の紹介と備えておきたい考え方について
Avatar for kazzpapa3 kazzpapa3
1
520
製造業ドメインにおける LLMプロダクト構築： 複雑な文脈へのアプローチ
Avatar for recruiting@caddi.jp caddi_eng
1
560
タスク管理も1on1も、もう「管理」じゃない ― KiroとBedrock AgentCoreで変わった"判断の仕事"
Avatar for Yusuke Shimizu yusukeshimizu
5
2.6k
DevOpsエージェントで実現する!！ AWS Well-Architected（W-A） を実現するシステム設計 / 20260307 Masaki Okuda
Avatar for SHIFT EVOLVE shift_evolve
PRO
3
610
IBM Bobを使って、PostgreSQLのToDoアプリをDb2へ変換してみよう/202603_Dojo_Bob
Avatar for Mayumi Hirano mayumihirano
1
320
Google系サービスで文字起こしから勝手にカレンダーを埋めるエージェントを作った話
Avatar for RisaTube risatube
0
150
親子 or ペアで Mashup for the Future！ しゃべって楽しむ 初手AI駆動でものづくり体験
Avatar for hiroramos hiroramos4
PRO
0
110
僕、S3 　シンプルって名前だけど全然シンプルじゃありません　よろしくお願いします
Avatar for Yuuki Yamashita yama3133
1
200
ナレッジワーク IT情報系キャリア研究セッション資料（情報処理学会 第88回全国大会 ）
Avatar for KNOWLEDGE WORK / 株式会社ナレッジワーク kworkdev
PRO
0
170

Featured

See All Featured
Product Roadmaps are Hard
Avatar for C. Todd Lombardo iamctodd
PRO
55
12k
[SF Ruby Conf 2025] Rails X
Avatar for Vladimir Dementyev palkan
2
820
The Myth of the Modular Monolith - Day 2 Keynote - Rails World 2024
Avatar for Eileen M. Uchitelle eileencodes
26
3.4k
sira's awesome portfolio website redesign presentation
Avatar for ElsiraPls elsirapls
0
190
Bridging the Design Gap: How Collaborative Modelling removes blockers to flow between stakeholders and teams @FastFlow conf
Avatar for Kenny Baas-Schwegler baasie
0
470
BBQ
Avatar for Matthew Crist matthewcrist
89
10k
Exploring anti-patterns in Rails
Avatar for Elle Meredith aemeredith
2
290
Principles of Awesome APIs and How to Build Them.
Avatar for Keavy McMinn keavy
128
17k
The browser strikes back
Avatar for Jono Alderson jonoalderson
0
780
Code Review Best Practice
Avatar for Trisha Gee trishagee
74
20k
Efficient Content Optimization with Google Search Console & Apps Script
Avatar for Katarina Dahlin katarinadahlin
PRO
1
400
The untapped power of vector embeddings
Avatar for Frank van Dijk frankvandijk
2
1.6k

Transcript

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

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

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

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

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

  6. 💡 解決策

  7. セルフレビュー😉

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

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

  10. やってみて感じたこと - 自分のコードに自信が持てるようになった - コードの質が上がった気がする - 変数・関数などの命名は適切か - 共通化できるところあるかどうか -

    分離してすっきりできそうなところがあるか - レビューがスムーズになった - セルフレビューをすることで PRの精度が高くなって、レビュアーや自分の負担が減る

  11. いいことたくさん👏

  12. おわり👋