threads / patch / 42962

patch, 3 partsdiff-highlight: add support for git log --graph output.

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

## tl;dr

2 messages between Jul 29, 2016 and Jul 29, 2016. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Brian Henderson· Jul 29, 2016, 14:51 UTC · lore
(resending as thread instead of attachments)

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.
 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
-- 
2.9.0
Jeff King· Jul 29, 2016, 16:40 UTC · re: Brian Henderson · lore

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

On Fri, Jul 29, 2016 at 07:51:27AM -0700, Brian Henderson wrote:
> (resending as thread instead of attachments)
Hmm, the actual patches don't seem to have made it to the list. :(
-Peff

← back to recent threads