Re: [PATCH v6 04/15] sequencer: introduce new commands to reset the revision

From: Sergey Organov <sorganov@gmail.com>
Date: Apr 11, 2018, 06:17 UTC
Message-ID: <878t9u5ksw.fsf@javad.com>
In-Reply-To: <fb797f32b7385317b9dc18524e3863ba627f6c98.1523362469.git.johannes.schindelin@gmx.de>

Hi Johannes,

Johannes Schindelin <johannes.schindelin@gmx.de> writes:

[...]

> We disallow '#' as label because that character will be used as separator
> in the upcoming `merge` command.

Please consider to use # not only in `merge` and `reset`, but in the rest of the commands as well, to unify this new syntax. I.e., right now it seems to be:

pick abcd A commit message merge beaf # B commit message

I suggest to turn it to:

pick abcd # A commit message merge beaf # B commit message

So that the # is finally universally the start of comment.

While we are at this, I couldn't find any even semi-formal syntax description of the entire todo list. Is there one already? Are you going to provide one?

-- Sergey

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