<p dir="ltr">So, a reason not to document this filter is if we haven't decided to formally freeze at our encoding.</p>
<p dir="ltr">I'm not sure if we have, and I don't know if we have enough tests for its input/output. If we do, then, it's OK to freeze this (if we're confident in the encoding).</p>
<div class="gmail_quote">On Mar 11, 2016 8:39 AM, "Yuya Nishihara" <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, 9 Mar 2016 17:03:30 +0000, Pierre-Yves David wrote:<br>
> On 03/07/2016 03:11 PM, Pierre-Yves David wrote:<br>
> > On 02/23/2016 08:35 PM, Augie Fackler wrote:<br>
> >> On Wed, Feb 24, 2016 at 12:45:32AM +0900, Yuya Nishihara wrote:<br>
> >>> # HG changeset patch<br>
> >>> # User Yuya Nishihara <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>><br>
> >>> # Date 1452939833 -32400<br>
> >>> #      Sat Jan 16 19:23:53 2016 +0900<br>
> >>> # Node ID 02072a6dba91478d5f12fd52113dce45f0b2a8c3<br>
> >>> # Parent  484371c3f9c61fd2694aae90b1c650dec203c596<br>
> >>> templatefilters: document json filter that requires UTF-8 bytes<br>
> >><br>
> >> I've queued patches 1-6, but I have a fuzzy memory that mpm had a<br>
> >> reason to not document json. Matt?<br>
> ><br>
> > Gentle ping.<br>
><br>
> According to timeless, we don't want to document this filter because it<br>
> would freeze the json format, something we are not ready to. I'm<br>
> dropping the patch from patchwork.<br>
<br>
I don't think this is related to timeless' work. "|json" filter is just<br>
a general-purpose JSON serializer. But I'm okay to go without a public<br>
docstring now. Instead, I'll add a comment that says input text should be<br>
UTF-8.<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org">Mercurial-devel@mercurial-scm.org</a><br>
<a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div>