[PATCH 1 of 3] context: add hiddenstr
Sean Farley
sean.michael.farley at gmail.com
Tue Jan 6 19:40:04 CST 2015
Pierre-Yves David writes:
> On 01/06/2015 03:31 PM, Sean Farley wrote:
>> I also thought so at first but the reason I suggested this route is
>> because:
>>
>> 1) the precedence by ctx.phasestr()
>
> The main reason for phasestr() is to avoid import to the phase module.
> This does not really apply here.
Sure but the method is still there. It'd be weird to have ctx.phasestr
and somethingelse.obsoletestr. Arguably, these methods should be in one
place. I think they should all go into the template engine but that is a
monumental amount of work.
More information about the Mercurial-devel
mailing list