Mercurial > hg > mercurial-crew-with-dirclash
comparison tests/test-hook @ 2221:05b6c13f43c6
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
author | Vadim Gelfer <vadim.gelfer@gmail.com> |
---|---|
date | Mon, 08 May 2006 10:59:58 -0700 |
parents | cbd458228a96 |
children | 4f072bb06e89 |
comparison
equal
deleted
inserted
replaced
2220:6d3cc2a982f3 | 2221:05b6c13f43c6 |
---|---|
98 a = list(args.items()) | 98 a = list(args.items()) |
99 a.sort() | 99 a.sort() |
100 print 'hook args:' | 100 print 'hook args:' |
101 for k, v in a: | 101 for k, v in a: |
102 print ' ', k, v | 102 print ' ', k, v |
103 return True | |
104 | 103 |
105 def passhook(**args): | 104 def passhook(**args): |
106 printargs(args) | 105 printargs(args) |
107 return True | |
108 | 106 |
109 def failhook(**args): | 107 def failhook(**args): |
110 printargs(args) | 108 printargs(args) |
109 return True | |
111 | 110 |
112 class LocalException(Exception): | 111 class LocalException(Exception): |
113 pass | 112 pass |
114 | 113 |
115 def raisehook(**args): | 114 def raisehook(**args): |