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

Re: [PATCH 0/3] diff-highlight: add support for git log --graph output.

From
Jakub Narębski <jnareb@gmail.com>
Date
Jul 29, 2016, 07:46 UTC
Message-ID
<579B09DD.9000603@gmail.com>
In-Reply-To
<20160728162712.GA29220@tci.corp.yp.com>
W dniu 2016-07-28 o 18:27, Brian Henderson pisze:
Show 19 quoted lines
> Hi, I've been working with Jeff King on this patch, but he encouraged me to
> email the list.
> 
> I recently learned about the diff-highlight tool and find it very helpful,
> however, I frequently use the --graph option to git log which breaks the tool.
> This patch looks to fix this.
> 
> I wanted to try and add some tests as well, but I was unsure what number to
> pick for the second digit. As there were limited tests in the contrib directory
> (only t93xx and one t7900) I just chose the next number in the 9xxx range.
> Please let me know if I need to change it.
> 
> I'm also not super happy about my test case for the graph option. If anyone has
> any better ideas, please let me know!
> 
> Brian Henderson (3):
>   diff-highlight: add some tests.
>   diff-highlight: add failing test for handling --graph output.
>   diff-highlight: add support for --graph output.

Could you please follow SubmittingPatches, and send patches in separate emails, inline, with those emails being replies to cover letter (like `git format-patch` and `git send-email` would do), and not as attachments to cover letter?

With email as it stands it is really difficult to review individual patches.

Show 12 quoted lines
> 
>  contrib/diff-highlight/Makefile                  |   5 +
>  contrib/diff-highlight/diff-highlight            |  13 +--
>  contrib/diff-highlight/t/Makefile                |  19 ++++
>  contrib/diff-highlight/t/t9400-diff-highlight.sh |  76 ++++++++++++++++
>  contrib/diff-highlight/t/test-diff-highlight.sh  | 111 +++++++++++++++++++++++
>  5 files changed, 218 insertions(+), 6 deletions(-)
>  create mode 100644 contrib/diff-highlight/Makefile
>  create mode 100644 contrib/diff-highlight/t/Makefile
>  create mode 100644 contrib/diff-highlight/t/t9400-diff-highlight.sh
>  create mode 100644 contrib/diff-highlight/t/test-diff-highlight.sh
> 
Previous: Brian Henderson
Message 2 of 2 in “diff-highlight: add support for git log --graph output.”
  1. 0/3 diff-highlight: add support for git log --graph output.Brian Henderson, Jul 28, 2016
  2. Jakub NarębskiJul 29, 2016

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.