[PATCH] churn: use ui.progress instead of --progress

Thu Mar 11 17:31:19 CST 2010

On 11 Mar 2010, at 17:31, Augie Fackler wrote:

> progress.disable = <bool> rather than a magic int, but yeah :)

Isn't that naming a bit backwards? In my opinion, having either ‘progress.enabled’ or ‘progress.show’ — defaulting to true — seem more logical choices. As long as the extension is separate from core, the option could simply default to ‘true’.

I hope I'm not nitpicking too much :)

--

Dan Villiom Podlaski Christiansen
danchr at gmail.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 1943 bytes
Desc: not available
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20100312/ca002d05/attachment.bin>