[PATCH 09 of 11] sslutil: use CA loaded state to drive validation logic

Gregory Szorc gregory.szorc at gmail.com
Thu May 5 17:26:55 EDT 2016


On Thu, May 5, 2016 at 1:31 AM, timeless <timeless at gmail.com> wrote:

> On Thu, May 5, 2016 at 3:53 AM, Gregory Szorc <gregory.szorc at gmail.com>
> wrote:
> > Right now, sslkwargs may set web.cacerts=! to indicate
>
> The style should be to document what the result of the commit is in
> present tense.
> So "Until now" would be better.
>
> > Since we no longer have a need for setting web.cacerts=!
> > in sslkwargs, we remove that.
>
> Once you've fixed that, this second bit makes more sense.
>

If this is the only thing wrong with the series, I'm going to hold off
sending a v2 so I don't spam inboxes. Perhaps the grammar police can
correct things in flight instead :)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20160505/20437ca7/attachment.html>


More information about the Mercurial-devel mailing list