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

Code Review with GitHub

Sponsored · Your Podcast. Everywhere. Effortlessly. Share. Educate. Inspire. Entertain. You do you. We'll handle the rest. →
Avatar for Uyi Ehondor Uyi Ehondor
June 18, 2013
Programming
0
160

Code Review with GitHub

Avatar for Uyi Ehondor

Uyi Ehondor

June 18, 2013
Tweet

Other Decks in Programming

See All in Programming
AtCoder Conference 2025
Avatar for shindannin shindannin
0
1k
20260127_試行錯誤の結晶を1冊に。著者が解説 先輩データサイエンティストからの指南書 / author's_commentary_ds_instructions_guide
Avatar for nash_efp nash_efp
0
930
2026年 エンジニアリング自己学習法
Avatar for yumechi(Motoki Hirao) yumechi
0
130
Basic Architectures
Avatar for Denys Poltorak denyspoltorak
0
660
[KNOTS 2026登壇資料]AIで拡張‧交差する プロダクト開発のプロセス および携わるメンバーの役割
Avatar for hisatake hisatake
0
250
Data-Centric Kaggle
Avatar for ねぼすけAI isax1015
2
770
Package Management Learnings from Homebrew
Avatar for Mike McQuaid mikemcquaid
0
210
CSC307 Lecture 07
Avatar for Javier Gonzalez-Sanchez javiergs
PRO
0
550
CSC307 Lecture 06
Avatar for Javier Gonzalez-Sanchez javiergs
PRO
0
680
組織で育むオブザーバビリティ
Avatar for ryotaro kobayashi ryota_hnk
0
170
Vibe Coding - AI 驅動的軟體開發
Avatar for Micky Li mickyp100
0
170
dchart: charts from deck markup
Avatar for Anthony Starks ajstarks
3
990

Featured

See All Featured
SEOcharity - Dark patterns in SEO and UX: How to avoid them and build a more ethical web
Avatar for Sara Fernández Carmona sarafernandez
0
110
Code Review Best Practice
Avatar for Trisha Gee trishagee
74
20k
Unsuck your backbone
Avatar for Amy Palamountain ammeep
671
58k
Noah Learner - AI + Me: how we built a GSC Bulk Export data pipeline
Avatar for Tech SEO Connect techseoconnect
PRO
0
100
Java REST API Framework Comparison - PWX 2021
Avatar for Matt Raible mraible
34
9.1k
How to Grow Your eCommerce with AI & Automation
Avatar for Katarina Dahlin katarinadahlin
PRO
0
100
How Software Deployment tools have changed in the past 20 years
Avatar for Geshan Manandhar geshan
0
32k
Avoiding the “Bad Training, Faster” Trap in the Age of AI
Avatar for Mike Taylor tmiket
0
72
Documentation Writing (for coders)
Avatar for Carmen Chung carmenintech
77
5.2k
How to build a perfect <img>
Avatar for Jono Alderson jonoalderson
1
4.9k
RailsConf & Balkan Ruby 2019: The Past, Present, and Future of Rails at GitHub
Avatar for Eileen M. Uchitelle eileencodes
141
34k
Believing is Seeing
Avatar for Spiro Bolos oripsolob
1
53

Transcript

  1. Sources: https://help.github.com/articles/using-pull-requests https://speakerdeck.com/takai/code-review-in-action Code Review with GitHub

  2. source: http://octodex.github.com/gangnamtocat/

  3. Collaborative Dev. Models • Fork & Pull • Shared Repository

  4. Fork & Pull Model • Popular choice with open source

    projects. • Encourages independence almost immediately without upfront coordination.

  5. Shared Repository Model • Prevalent with small teams and organisations

    collaborating on private projects.

  6. 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.

  7. How To: Pull Requests See "More Reading"

  8. Why Code Review • Improved code quality • Fewer defects

    in code • Improved communication about code content • ...

  9. Good Code Review • What does not meet set standards?

    • Why it does not meet standards? • How to improve • ...

  10. More Reading https://help.github.com/articles/using-pull-requests

  11. Clean code depends on... YOU

  12. Q & A