Re: [PATCH 0/5] refs: remove functions without ref store

From: Junio C Hamano <gitster@pobox.com>
Date: May 3, 2024, 19:35 UTC
Message-ID: <xmqqttje6646.fsf@gitster.g>
In-Reply-To: <ZjUzyNnBW1BAtEc/@nand.local>

Taylor Blau <me@ttaylorr.com> writes:

> But TBH I think this is probably overkill and anybody who encounters an
> issue like this likely does not need the extra hand.

As long as the replacement is clearly obvious. Is it always the rule that "when the function F you tried to call is missing, just prefix F with refs_ and add the first parameter that is a ref_store, i.e., refs_f(get_main_ref_store(the_repository), ...)"? Even if that were the case, some in-header comment would help the affected topic owners than having nothing at all.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.