Re: [PATCH v4 0/2] Conditional config includes based on remote URL

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

Subject: Re: [PATCH v4 0/2] Conditional config includes based on remote URL
From: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
Date: Mon, 29 Nov 2021 21:48:16 +0100
Cc: git@xxxxxxxxxxxxxxx, chooglen@xxxxxxxxxx, gitster@xxxxxxxxx
In-reply-to: <cover.1638217387.git.jonathantanmy@google.com>
References: <cover.1634077795.git.jonathantanmy@google.com> <cover.1638217387.git.jonathantanmy@google.com>
User-agent: Debian GNU/Linux bookworm/sid; Emacs 27.1; mu4e 1.6.9

On Mon, Nov 29 2021, Jonathan Tan wrote:

> Thanks everyone for your comments. Here's an update.

Just from skimming this (minor) feedback on v3 still applies:
https://lore.kernel.org/git/211123.86pmqrwtf2.gmgdl@xxxxxxxxxxxxxxxxxxx/

I.e. s/hasremoteurl/hasRemoteURL/ etc. in appropriate places.

Follow-Ups:
- Re: [PATCH v4 0/2] Conditional config includes based on remote URL
  - From: Junio C Hamano

References:
- [RFC PATCH 0/2] Conditional config includes based on remote URL
  - From: Jonathan Tan
- [PATCH v4 0/2] Conditional config includes based on remote URL
  - From: Jonathan Tan

Prev by Date: [PATCH v4 1/2] config: make git_config_include() static
Next by Date: [PATCH v2 0/6] Allow writing invalid OIDs into refs for testing purposes
Previous by thread: [PATCH v4 1/2] config: make git_config_include() static
Next by thread: Re: [PATCH v4 0/2] Conditional config includes based on remote URL
Index(es):
- Date
- Thread

[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