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

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

Sponsored · Your Podcast. Everywhere. Effortlessly. Share. Educate. Inspire. Entertain. You do you. We'll handle the rest. →
Avatar for ishitoko shiho ishitoko shiho
April 20, 2023
Technology
1.8k
0

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

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

Avatar for ishitoko shiho

ishitoko shiho

April 20, 2023

More Decks by ishitoko shiho

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

Other Decks in Technology

See All in Technology
NDIAS CTF 2026 問題解説会資料
Avatar for bata_24 bata_24
0
180
なぜ私たちのSREプラクティスはなかなか機能しないのか 〜システムより先に組織を見る〜 / Why our SRE practices aren't really working
Avatar for VTRyo vtryo
0
800
4人目のSREはAgent
Avatar for tanimuyk tanimuyk
0
390
AIで政治は変わるのか？ — 中高生と考えたAI時代の民主主義（東海高校サタデープログラム）
Avatar for 須田英太郎 eitarosuda
0
390
型は壁、Rustでもバグを直すな、表現できなくせよ
Avatar for nwiizo nwiizo
11
1.4k
AIDLC_ヤフーショッピングの取り組み
Avatar for LINEヤフーTech (LY Corporation Tech) lycorptech_jp
PRO
0
530
Empower GenAI with Agile - あなたのアジャイルが生成AIのバフになる仕組み
Avatar for Hiroyuki Ito (TheHiro) hageyahhoo
0
110
知見・人・API・DB・予算 ─ ナイナイ尽くしだった人事データ整備 with dbt、5年間の学び
Avatar for ken ken6377
1
150
從觀望到全公司落地：AI Agentic Coding 導入實戰 — 流程整合與安全治理
Avatar for Bo-Yi Wu appleboy
1
940
CSに"SLO"は要らない、経営層に"99.9%"は伝わらない - SREを全社に"翻訳"する3原則
Avatar for サイバーセキュリティクラウド cscengineer
PRO
0
1.7k
Amazon EVS で VCF 9.0 / 9.1 のサポート開始まとめ
Avatar for mtoyoda mtoyoda
0
250
Keeping applications secure by evolving OAuth 2.0 and OpenID Connect
Avatar for Alexander Schwartz ahus1
PRO
1
140

Featured

See All Featured
Music & Morning Musume
Avatar for Bryan Veloso bryan
47
7.3k
WENDY [Excerpt]
Avatar for Tessa Abrams tessaabrams
11
38k
Test your architecture with Archunit
Avatar for Yoan thirion
1
2.3k
Leo the Paperboy
Avatar for Maya Tellez mayatellez
8
1.9k
GraphQLとの向き合い方2022年版
Avatar for Yosuke Kurami quramy
50
15k
Building Better People: How to give real-time feedback that sticks.
Avatar for Will Jessup wjessup
370
20k
Applied NLP in the Age of Generative AI
Avatar for Ines Montani inesmontani
PRO
4
2.4k
Building the Perfect Custom Keyboard
Avatar for Naoto Takai takai
2
810
The State of eCommerce SEO: How to Win in Today's Products SERPs - #SEOweek
Avatar for Aleyda Solis aleyda
2
11k
How to train your dragon (web standard)
Avatar for Monica Dinculescu notwaldorf
97
6.7k
Documentation Writing (for coders)
Avatar for Carmen Chung carmenintech
77
5.4k
Thoughts on Productivity
Avatar for Jon Yablonski jonyablonski
76
5.2k

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. おわり👋