From: Junio C Hamano Date: Wed, 09 Jun 2021 09:30:25 GMT Subject: Re: [PATCH v2 1/4] promisor-remote: read partialClone config here Message-ID: In-Reply-To: <20210609042649.2322758-1-jonathantanmy@google.com> Jonathan Tan writes: > I'm reluctant to add more fields to struct repository_format. I am only suggesting to add one new member to either struct repository or struct repo_settings, so that it becomes crystal clear that struct repository_format is about each single repository, not the global the_repository. Other things that partial repository support needs to keep *and* do not directly come from extensions would not belong to repository_format and should not be added there, but what we read from extensions.* for each repository belongs to each instance of in-core repository structure and should be discoverable starting from "struct repository", no?