Today, I witnessed the reason for doing a patch review. Two days back, I landed a load of changes to the Evolution-sharp cvs along with a stupid change to one of the functions in the calendar glue-code that would cause, actually, caused a pretty straight "crash".
Though, my unit-testing passed with out any error, the crash was very obvious when Joe pinged me on the channel.
This blog would keep reminding me to do a self-patch-review before commits.
Also, a message to myself, "DO NOT COMMIT ANYTHING PAST MIDNIGHT."
Subscribe to:
Post Comments (Atom)
No comments:
Post a Comment