Code review
-
Upload
maciej-malarz -
Category
Technology
-
view
115 -
download
0
Transcript of Code review
Code Review
Maciej Malarz (@malarzm)
@malarzm
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Code Review
Maciej Malarz (@malarzm)
Pull Requestaka
Merge Requestaka...
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Communication
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Bus Factor
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Tech Debt
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
++Code Quality
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
So the feature is done and working BUT I wouldn't merge my
own code so I'll refactor it first.
Another pair of eyes
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Check Logic
Maciej Malarz (@malarzm)
Proofread
Maciej Malarz (@malarzm)
ProofreadCheck Logic
Sharing Responsibility
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
<alcaeus> Looks like markup is broken in the blog post<beberlei> amateurs ;)<alcaeus> hey, you merged broken stuff :P<beberlei> you +1ed it, i wouldnt do that again :P<malarzm> blargh
Learning
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
I'll merge the PR but only if you tell me what does these
things do.
Proof of Concept
Maciej Malarz (@malarzm)
History!
Maciej Malarz (@malarzm)
Anything else?
Maciej Malarz (@malarzm)
How to conduct?
Maciej Malarz (@malarzm)
Reread your changes
Maciej Malarz (@malarzm)
Be kind!
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
In doubt? Ask!
Maciej Malarz (@malarzm)
Have better idea?Tell!
Maciej Malarz (@malarzm)
GitHub goodies
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
One true measure
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
Thank you!
Maciej Malarz (@malarzm)
Questions?
Maciej Malarz (@malarzm)
Thank you!
Maciej Malarz (@malarzm)
Maciej Malarz (@malarzm)
http://www.animhut.com/freebies/vectors/free-vector-character-the-programmer/https://www.host.al/assets/img/avatar.pngTeam Geek by Ben Collins-Sussman, Brian W. Fitzpatrick Publisher: O'Reilly Media, Inc. Published: July 2012 Referennce ID 9781449329839http://www.furrytalk.com/tag/stuffing/http://clipartion.com/free-clipart-googly-eyes-clip-art/http://explosm.nethttp://commadot.com