Re: [PATCH 0/2] scalar: add --no-maintenance option
- From
Junio C Hamano <gitster@pobox.com>
- Date
- May 1, 2025, 18:20 UTC
- Message-ID
- <xmqq7c30xkdx.fsf@gitster.g>
- In-Reply-To
- <xmqqh624xp4j.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> writes:
> It is mostly philosophical, I think, ...
In other words, I think making each helper function do one thing and one thing well is a good thing, and "register" should focus on registering the new repository to the system.
By the way, it is excellent that the new option honors the positive form (i.e. "clone --maintenance" and "clone --no-maintenance"). I was re-reading the patches and was pleasantly surprised, as I would have forgotten to do so while focusing too much on the shiny new feature of being able to disable if I were doing these patches.