archive mirror
 help / color / mirror / Atom feed
Cc:,,, Jubilee Young <>
Subject: [Outreachy] [PATCH v7 0/1] Revenge of --bisect --first-parent
Date: Mon,  4 Nov 2019 21:21:40 -0800	[thread overview]
Message-ID: <> (raw)

From: Jubilee Young <>


I meant to introduce myself quite a bit earlier, but alas, the world is
the kind which also results in version control systems being critical
instead of merely desirable. At least, for software development. Yet,
time marches on. I know you have quite a few applicants to pick between
for the Outreachy internship but I would be honored if you would grant
me your consideration, nonetheless.

...And if you would merge this patch, too, if it's (finally) up to 
standard! This is the renovation and update of a long-abandoned feature
branch begun by Harald Nordgren and extended by Tiago Botelho some time
back. The world caught up with them, too, but one re(base|animation)
later, these works live on, zombie-like. I did my best to understand
and integrate everything with the code base in spite of the many
changes introduced since. There were many recommendations to the last
iteration, and a fair number of changes were integrated essentially as 
they were given, with the most extensive work being in the department
of the tests and refactoring them to reuse the originally generated
commit graph.

My cursory experiments seem to yield the desired result, and the tests
(which have already proven useful, to my dismay) do pass, so it seems
relatively sound for a first try. But it's possible I misunderstood
something and it still needs work. If so, please let me know so that I
can fix it and at long last, this patch can live again!

Bonus reading for context (most recent first):

Jubilee Young (1):
  Implement rev-list --bisect* --first-parent

 bisect.c                   | 56 ++++++++++++++++++++++++++------------
 bisect.h                   | 17 ++++++++++--
 builtin/rev-list.c         |  9 ++++--
 revision.c                 |  3 --
 t/   |  4 +--
 t/ | 54 ++++++++++++++++++++++++++++++++++++
 6 files changed, 116 insertions(+), 27 deletions(-)


             reply	other threads:[~2019-11-05  5:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-05  5:21 workingjubilee [this message]
2019-11-05  5:21 ` [PATCH v7 1/1] Implement rev-list --bisect* --first-parent workingjubilee
2019-11-05 23:04   ` Jonathan Tan
2019-11-06  2:42     ` Junio C Hamano
2019-11-06 11:30       ` Johannes Schindelin
2019-11-06 21:36       ` Jonathan Tan
2019-11-07  3:35         ` Junio C Hamano
2019-11-07  5:07         ` Jubilee Young
2019-11-07 16:00     ` Jubilee Young
2019-11-07 18:56       ` Jonathan Tan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \
    --subject='Re: [Outreachy] [PATCH v7 0/1] Revenge of --bisect --first-parent' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
on how to clone and mirror all data and code used for this inbox