Wednesday, August 03, 2005

Why a patch review is required?

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

No comments: