Re: [PATCH 3/3] add-interactive: add new "context" subcommand

From: Christian Couder <christian.couder@gmail.com>
Date: May 6, 2025, 08:28 UTC
Message-ID: <CAP8UFD00bzcrdLLXwjy5yUrvXpnjTDG6ohMTahgpziRD63udUw@mail.gmail.com>
In-Reply-To: <CAP9jKjFH_2CmvVAQwfk4KKTjDyMne2Wq9rkVnn+XB0dD5BEOCQ@mail.gmail.com>

On Tue, May 6, 2025 at 9:37 AM Leon Michalak <leonmichalak6@gmail.com> wrote:

>
> Valid points, I don't think I have any objections to anything listed.

Please don't "top post" and reply inline instead (see https://en.wikipedia.org/wiki/Posting_style) as it's the standard on this list.

> Would it be recommended to update to test_grep (and test_config from
> previous message) in the same test files whilst I'm at it?

If you want to replace pre existing "grep" commands with "test_grep" and pre existing `git config` with `test_config`, I would recommend doing it in preparatory patches that go first in your patch series.

Thanks!

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