Re: [PATCH] doc: fix config API documentation about config_with_options
- Date: Mon, 14 May 2018 11:22:25 -0700
- From: Brandon Williams <bmwill@xxxxxxxxxx>
- Subject: Re: [PATCH] doc: fix config API documentation about config_with_options
On 05/12, Antonio Ospite wrote:
> On Wed, 9 May 2018 10:19:50 -0700
> Brandon Williams <bmwill@xxxxxxxxxx> wrote:
> > On 05/09, Antonio Ospite wrote:
> > > In commit dc8441fdb ("config: don't implicitly use gitdir or commondir",
> > > 2017-06-14) the function git_config_with_options was renamed to
> > > config_with_options to better reflect the fact that it does not access
> > > the git global config or the repo config by default.
> > >
> > > However Documentation/technical/api-config.txt still refers to the
> > > previous name, fix that.
> > >
> > > While at it also update the documentation about the extra parameters,
> > > because they too changed since the initial definition.
> > >
> > > Signed-off-by: Antonio Ospite <ao2@xxxxxx>
> > > ---
> > >
> > > Patch based on the maint branch.
> > Thanks for updating the docs. Maybe one day we can migrate these docs
> > to the source files themselves, making it easier to keep up to date.
> > For now this is good :)
> Should I resend the patch to gitster@xxxxxxxxx with your Acked-by?
This has my Reviewed-by: Brandon Williams <bmwill@xxxxxxxxxx>
though you don't need to resend the patch to Junio, he has normally
taken care of that :)
> Antonio Ospite
> A: Because it messes up the order in which people normally read text.
> See http://en.wikipedia.org/wiki/Posting_style
> Q: Why is top-posting such a bad thing?