Lock in $30 Savings on PRO—Offer Ends Soon! ⏳
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
はじめてのプルリクエスト
Search
ishitoko shiho
April 20, 2023
Technology
0
1.6k
はじめてのプルリクエスト
ハードル激低LT大会ッ! #02 で発表したスライド
ishitoko shiho
April 20, 2023
Tweet
Share
More Decks by ishitoko shiho
See All by ishitoko shiho
はじめて tsgo について興味を持ったときに私の語ること
cidermitaina
0
870
Other Decks in Technology
See All in Technology
新 Security HubがついにGA!仕組みや料金を深堀り #AWSreInvent #regrowth / AWS Security Hub Advanced GA
masahirokawahara
1
1.6k
Kubernetes Multi-tenancy: Principles and Practices for Large Scale Internal Platforms
hhiroshell
0
120
AWS Bedrock AgentCoreで作る 1on1支援AIエージェント 〜Memory × Evaluationsによる実践開発〜
yusukeshimizu
6
380
技術以外の世界に『越境』しエンジニアとして進化を遂げる 〜Kotlinへの愛とDevHRとしての挑戦を添えて〜
subroh0508
1
420
手動から自動へ、そしてその先へ
moritamasami
0
290
意外とあった SQL Server 関連アップデート + Database Savings Plans
stknohg
PRO
0
300
Lessons from Migrating to OpenSearch: Shard Design, Log Ingestion, and UI Decisions
sansantech
PRO
1
110
SSO方式とJumpアカウント方式の比較と設計方針
yuobayashi
7
580
20251209_WAKECareer_生成AIを活用した設計・開発プロセス
syobochim
5
1.4k
Playwrightのソースコードに見る、自動テストを自動で書く技術
yusukeiwaki
13
5.2k
AWS Trainium3 をちょっと身近に感じたい
bigmuramura
1
130
ガバメントクラウド利用システムのライフサイクルについて
techniczna
0
190
Featured
See All Featured
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
52
5.7k
Unsuck your backbone
ammeep
671
58k
Six Lessons from altMBA
skipperchong
29
4.1k
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
marktimemedia
31
2.6k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
36
6.2k
YesSQL, Process and Tooling at Scale
rocio
174
15k
How GitHub (no longer) Works
holman
316
140k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
132
19k
Evolution of real-time – Irina Nazarova, EuRuKo, 2024
irinanazarova
9
1.1k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
47
7.9k
The Myth of the Modular Monolith - Day 2 Keynote - Rails World 2024
eileencodes
26
3.2k
StorybookのUI Testing Handbookを読んだ
zakiyama
31
6.4k
Transcript
はじめてのプルリクエスト @cidermitaina 2023.04.20 ハードル激低LT大会ッ! #02
@cidermitaina - フロントエンドエンジニア @SmartHR - UI作るのがすきです。 - ひとみしりです。
突然ですが、みなさん、
PRを作成してレビューをしてもらうとき、不安になったりし ませんか?🥺
😢 - たくさん指摘があったらどうしよう… - コメントがたくさんあると批判されてる気持ちになる… - 自分のコードに自信がない…
💡 解決策
セルフレビュー😉
セルフレビューとは? レビュー依頼を出す前に、自分でレビューをすること。
セルフレビューで私がやっていること - 1行1行、頭の中で説明しながら読んでいく - なんでそのコードを書いたのか説明できない箇所は理解できていない証拠。 - 自分のコードを客観的にレビューしてみる - 自分のコードではなく、他人のコードだと思ってレビューしてみる
やってみて感じたこと - 自分のコードに自信が持てるようになった - コードの質が上がった気がする - 変数・関数などの命名は適切か - 共通化できるところあるかどうか -
分離してすっきりできそうなところがあるか - レビューがスムーズになった - セルフレビューをすることで PRの精度が高くなって、レビュアーや自分の負担が減る
いいことたくさん👏
おわり👋