This document is posted to help you gain knowledge. Please leave a comment to let me know what you think about it! Share it to your friends and learn new things together.
Transcript
Peer Review
Ombu Labs, February 2015
Git Branches
masterIt’s production!
developIt’s the unstable branch
feature/twitter-integrationIt describes a feature
hotfixes/price-bugIt describes a bug fix
Pull Requests
TitleBrief summary of what it is
DescriptionLonger description of the changes
CommitsMake them as granular as possible
CodeChanges to the code
Test CoverageChanges that prove your changes work
Guidelines
Be niceDon’t be mean, we are all learning
Don’t take it personalAny code can be improved
KISSSolve a concrete problem now
RefactorYou can always refactor later
Don’t design for the future
You’re not an oracle, neither am I
DocumentAny public interface should be documented (YARD)
Check GuidelinesWe should all follow our style guidelines