[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