git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Extending whitespace checks

From
KHKristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date
Nov 24, 2024, 21:58 UTC
Message-ID
<89b1b39c-a6d2-4f63-9cc5-722772bddd8a@app.fastmail.com>
In-Reply-To
<D5UQHS9IV5N1.3IO1848Q1730B@ferdinandy.com>
On Sun, Nov 24, 2024, at 22:41, Bence Ferdinandy wrote:
Show 9 quoted lines
> This might be a tangent, but since enhancing whitespace checking was mentioned,
> I'd thought I note here:  `git log --check` running in the CI did not catch the
> white space errors in this patch (see the last hunk):
>
> https://lore.kernel.org/git/20241121225757.3877852-4-bence@ferdinandy.com/
>
> although it would have been certainly nice. I'm not sure if --check could
> already catch this actually, or if it would be easy/possible to have something
> general enough that does catch it.

It looks like you indented some lines with spaces instead of tabs. It doesn’t look like a “whitespace error” in the `--check` sense as I understand it.

Previous: Bence FerdinandyNext: A bughunter
Message 3 of 9 in “Extending whitespace checks”
  1. Junio C HamanoNov 24, 2024
  2. Bence FerdinandyNov 24, 2024
  3. Kristoffer HaugsbakkNov 24, 2024
  4. A bughunterDec 1, 2024
  5. Junio C HamanoNov 25, 2024
  6. Jacob KellerNov 25, 2024
  7. Jeff KingNov 27, 2024
  8. Junio C HamanoNov 27, 2024
  9. Jeff KingDec 1, 2024

Read the whole thread, see it on lore, or plain text.

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