Re: [PATCH v2 2/3] builtin/rebase.c: Emit warning when rebasing without a forkpoint

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 5, 2023, 22:01 UTC
Message-ID: <xmqqfs3ss2i1.fsf@gitster.g>
In-Reply-To: <2e1f1d2a-4aec-4b60-bc96-685a27055c06@opperschaap.net>

Wesley Schwengle <wesleys@opperschaap.net> writes:

> I like the idea of the warning, but it could be loud indeed and you'll
> want to turn it off in that case.

I tend to think that a single-liner warning would not be too intrusive (it might actually be too subtle to be noticed), especially given that it is issued only when the fork-point does move the target commit from what was given.

Giving a shortlog of what is lost in the history does sound like a bit too loud, I am afraind, though.

$ 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.