Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
Code Review with GitHub
Search
Sponsored
·
SiteGround - Reliable hosting with speed, security, and support you can count on.
→
Uyi Ehondor
June 18, 2013
Programming
0
160
Code Review with GitHub
Uyi Ehondor
June 18, 2013
Tweet
Share
Other Decks in Programming
See All in Programming
humanlayerのブログから学ぶ、良いCLAUDE.mdの書き方
tsukamoto1783
0
180
Apache Iceberg V3 and migration to V3
tomtanaka
0
150
メルカリのリーダビリティチームが取り組む、AI時代のスケーラブルな品質文化
cloverrose
2
510
CSC307 Lecture 09
javiergs
PRO
1
830
副作用をどこに置くか問題:オブジェクト指向で整理する設計判断ツリー
koxya
1
590
今こそ知るべき耐量子計算機暗号(PQC)入門 / PQC: What You Need to Know Now
mackey0225
3
370
The Past, Present, and Future of Enterprise Java
ivargrimstad
0
490
MUSUBIXとは
nahisaho
0
130
登壇資料を作る時に意識していること #登壇資料_findy
konifar
4
950
AI Agent Tool のためのバックエンドアーキテクチャを考える #encraft
izumin5210
6
1.8k
高速開発のためのコード整理術
sutetotanuki
1
390
AIエージェント、”どう作るか”で差は出るか? / AI Agents: Does the "How" Make a Difference?
rkaga
4
2k
Featured
See All Featured
Optimising Largest Contentful Paint
csswizardry
37
3.6k
Statistics for Hackers
jakevdp
799
230k
Leading Effective Engineering Teams in the AI Era
addyosmani
9
1.6k
SEO Brein meetup: CTRL+C is not how to scale international SEO
lindahogenes
0
2.3k
B2B Lead Gen: Tactics, Traps & Triumph
marketingsoph
0
52
The Language of Interfaces
destraynor
162
26k
Lightning talk: Run Django tests with GitHub Actions
sabderemane
0
110
Ruling the World: When Life Gets Gamed
codingconduct
0
140
Redefining SEO in the New Era of Traffic Generation
szymonslowik
1
210
Breaking role norms: Why Content Design is so much more than writing copy - Taylor Woolridge
uxyall
0
160
DevOps and Value Stream Thinking: Enabling flow, efficiency and business value
helenjbeal
1
89
A Modern Web Designer's Workflow
chriscoyier
698
190k
Transcript
Sources: https://help.github.com/articles/using-pull-requests https://speakerdeck.com/takai/code-review-in-action Code Review with GitHub
source: http://octodex.github.com/gangnamtocat/
Collaborative Dev. Models • Fork & Pull • Shared Repository
Fork & Pull Model • Popular choice with open source
projects. • Encourages independence almost immediately without upfront coordination.
Shared Repository Model • Prevalent with small teams and organisations
collaborating on private projects.
Pull Requests • Pull requests let you tell others about
changes. • Interested parties will be notified. • Parties can review the set of changes, discuss potential modifications, and even push follow-up commits if necessary.
How To: Pull Requests See "More Reading"
Why Code Review • Improved code quality • Fewer defects
in code • Improved communication about code content • ...
Good Code Review • What does not meet set standards?
• Why it does not meet standards? • How to improve • ...
More Reading https://help.github.com/articles/using-pull-requests
Clean code depends on... YOU
Q & A