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

誰が為のPull Requestか？

Avatar for Hiroya Hinomori(KTC) Hiroya Hinomori(KTC)
May 15, 2024
0
100

誰が為のPull Requestか？

Avatar for Hiroya Hinomori(KTC)

Hiroya Hinomori(KTC)

May 15, 2024
Tweet

More Decks by Hiroya Hinomori(KTC)

See All by Hiroya Hinomori(KTC)
iOSDC初出展までにした事を共有したい
Avatar for Hiroya Hinomori(KTC) ktchiroyah
0
100
エンジニアが開発生産性を上げるためにやる最初の一歩
Avatar for Hiroya Hinomori(KTC) ktchiroyah
0
310
内製化でクルマの未来を作るぼくらの話
Avatar for Hiroya Hinomori(KTC) ktchiroyah
0
81

Featured

See All Featured
StorybookのUI Testing Handbookを読んだ
Avatar for Shingo Yamazaki zakiyama
30
6k
Embracing the Ebb and Flow
Avatar for Simon Collison colly
87
4.8k
Save Time (by Creating Custom Rails Generators)
Avatar for Garrett Dimon garrettdimon
PRO
32
1.4k
Why Our Code Smells
Avatar for Brandon Keepers bkeepers
PRO
338
57k
The Art of Delivering Value - GDevCon NA Keynote
Avatar for David Neal reverentgeek
15
1.6k
Sharpening the Axe: The Primacy of Toolmaking
Avatar for Bryan Cantrill bcantrill
44
2.5k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
Avatar for Kevin Liebholz kevinliebholz
34
6k
Typedesign – Prime Four
Avatar for Hannes Fritz hannesfritz
42
2.8k
Understanding Cognitive Biases in Performance Measurement
Avatar for Philip Tellis bluesmoon
29
1.9k
ReactJS: Keep Simple. Everything can be a component!
Avatar for Pedro Nauck pedronauck
667
120k
GitHub's CSS Performance
Avatar for Jon Rohan jonrohan
1031
460k
No one is an island. Learnings from fostering a developers community.
Avatar for Antonio thoeni
21
3.4k

Transcript

  1. 誰が為の Pull Requestか？ モバイル開発グループ 日野森 寛也

  2. みなさん

  3. Pull Request をお願いする時

  4. 自分が書いたコードを ちゃんと説明できてますか？

  5. コード書いたんだから見てよ！ 急いでるから早くApproveくださいいい！

  6. Point 1 PRの中に入れる変更内容は そのPRに沿った内容になっているか？

  7. Point2 難しい仕様や、伝わりづらそうな実装内容に コメントを添える

  8. Point3 PRのテンプレートをうまく活用していきましょう

  9. 🍶