I always review the local diff before pushing. Can sometimes catch typos, or unc... | Hacker News

Hacker Newsnew | past | comments | ask | show | jobs | submit

		blub 87 days ago \| parent \| context \| favorite \| on: At Amazon, some coders say their jobs have begun t... I always review the local diff before pushing. Can sometimes catch typos, or unclear comments or naming issues. The concept and design were by that point iterated on, so it doesn’t happen that I need to rewrite a significant amount of code.

strogonoff 86 days ago [–]

My preferred workflow requires me to go through every changed chunk and stage them one by one. It’s very easy with vim-fugitive. To keep commits focused, it requires reading every chunk, which I guess is an implicit review of sorts.

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact