reverse sense of return value from python hooks. old scheme (False/None/0/'' == fail) made coding style unnatural, did not allow use of mercurial commands as hooks. new scheme (False/None/0 == pass) is pythonic, does not require peculiar "return True" at ends of hooks, allows hooks like this: [hooks] # update working dir after push into this repo changegroup.update = python:mercurial.commands.update

--- a/hgext/notify.py
+++ b/hgext/notify.py
@@ -255,4 +255,3 @@ def hook(ui, repo, hooktype, node=None, 
         n.node(node)
     n.diff(node)
     n.send(node, count)
-    return True