Code review comment for lp:~torkvemada/bzr/commit_hooks

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

This doesn't look unreasonable.

I'm wary about passing an instance of the Commit object to a hook; Commit at the moment is fairly private, exposing it this way makes it harder to change things there in the future.

« Back to merge proposal