Re: [PATCH] config: support remote name in includeIf.hasconfig condition

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sunday, October 20th, 2024 at 11:33 AM, Kristoffer Haugsbakk <kristofferhaugsbakk@xxxxxxxxxxxx> wrote:

> 
> 
> Hi
> 
> On Sun, Oct 20, 2024, at 17:23, Ken Matsui wrote:
> 
> > Before this patch, includeIf.hasconfig only accepted remote.*.url,
> 
> 
> The conventional description of the current state of the code, without
> this patch, is the present tense. What the patch does uses the
> imperative mood. E.g.
> 
> includeIf.hasconfig only accepts remote.*.url
> 
> See Documentation/SubmittingPatches, “present-tense” and
> “imperative-mood”.

Oops, thank you for pointing this out.  I will update the
description and send an updated patch.

> 
> --
> Kristoffer Haugsbakk





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux