Thread (9 messages) 9 messages, 3 authors, 2024-11-06

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

From: Ken Matsui <hidden>
Date: 2024-10-20 15:43:31

On Sunday, October 20th, 2024 at 11:33 AM, Kristoffer Haugsbakk [off-list ref] wrote:

Hi

On Sun, Oct 20, 2024, at 17:23, Ken Matsui wrote:
quoted
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help