[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [PATCH 0/2] add notes.merge configuration variable
From:       Junio C Hamano <gitster () pobox ! com>
Date:       2015-07-31 20:00:10
Message-ID: xmqqvbd03xo5.fsf () gitster ! dls ! corp ! google ! com
[Download RAW message or body]

CC'ing yourself is ok, but please do not forget to ask reviews from
people who have touched the area of the code you are touching (they
are hopefully the ones who are more knowledgeable and can help you).

Old timers like I know Johan is the go-to person on things around
notes, but new people can ask their copy of Git:

$ git shortlog -n -s --no-merges master builtin/notes.c notes.c | head -n 5
    46	Johan Herland
    11	Junio C Hamano
    11	Michael Haggerty
     5	Jeff King
     5	Nguyễn Thái Ngọc Duy

The changes to the code looked reasonable from a cursory read,
though.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]