Re: [PATCH v4 05/21] check-attr: there are only two possible line terminations

From: Jeff King <peff@peff.net>
Date: Jan 15, 2016, 21:23 UTC
Message-ID: <20160115212300.GB29698@sigill.intra.peff.net>
In-Reply-To: <xmqq60yuh74w.fsf@gitster.mtv.corp.google.com>

On Fri, Jan 15, 2016 at 01:20:31PM -0800, Junio C Hamano wrote:

Show 9 quoted lines

> Jeff King <peff@peff.net> writes:
> 
> > Apparently we use this pattern in quite a few places (or they all copied
> > from each other). Here's a possible fixup that could go after your
> > series (it could go before, too, but it creates a lot of textual
> > conflicts with your stuff).
> 
> Sounds good, but I'm in favor of leaving this as an unrelated
> clean-up that happens to touch the same area.

Would you prefer me to hold it and send it later, after your series is merged, or just rebase it on master, have it graduate separately, and let you deal with the conflicts?

-Peff

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