git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Duy Nguyen <pclouds@gmail.com>
Cc: "Jason St. John" <jstjohn@purdue.edu>,
	Christian Couder <christian.couder@gmail.com>,
	Michael J Gruber <git@drmicha.warpmail.net>,
	David Kastrup <dak@gnu.org>, git <git@vger.kernel.org>
Subject: Re: Promoting Git developers
Date: Tue, 10 Mar 2015 21:16:02 -0700	[thread overview]
Message-ID: <xmqqd24g6uf1.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <CACsJy8CHmdSRTfspKfSqtg7VXT7D6uxqr49KQQe8dhE5popakg@mail.gmail.com> (Duy Nguyen's message of "Wed, 11 Mar 2015 09:13:40 +0700")

Duy Nguyen <pclouds@gmail.com> writes:

> ... We may want to acknowledge review efforts as well, by
> grepping Helped-by:, Reviewed-by:...

Agreed. Something along the lines of 

    $ git shortlog --no-merges -s -n -t Helped-by -t Reviewed-by v2.3.0..
       6  4  0  Michael Haggerty
       3  0  1  Jeff King
       3  2  1  Junio C Hamano
       1  0  0  Anders Kaseorg
       1  0  0  Ben Walton
       1  0  0  Jean-Noel Avila
       1  0  0  Michael J Gruber
       1  0  0  Michal Sojka
       1  0  0  Mikko Rapeli
       1  0  0  Mårten Kongstad
       1  0  0  Nguyễn Thái Ngọc Duy
       1  0  7  Stefan Beller

that gives the number of trailer entries specified with -t in the
order specified when doing the short/abbreviated form may be a good
thing to have. The output can be piped to "sort -k" and "cut" to be
cooked in any way.

For completeness, in the long form, the extra numbers probably would
come next to names of the individual:

    $ git shortlog --no-merges -n -t Helped-by -t Reviewed-by v2.3.0..
    Michael Haggerty (6, 4, 0):
          write_ref_sha1(): remove check for lock == NULL
          write_ref_sha1(): move write elision test to callers
          lock_ref_sha1_basic(): do not set force_write for missing references
          reflog: improve and update documentation
          reflog_expire(): ignore --updateref for symbolic references
          reflog_expire(): never update a reference to null_sha1

    Jeff King (3, 0, 1):
          gettext.c: move get_preferred_languages() from http.c
          diffcore-rename: split locate_rename_dst into two functions
          diffcore-rename: avoid processing duplicate destinations
    ...

The long format needs to be careful not to drop those who helped
others without any commit under their own names.

  reply	other threads:[~2015-03-11  4:16 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-07  7:18 Promoting Git developers (was: Bashing freelancers) Christian Couder
2015-03-09 13:57 ` Michael J Gruber
2015-03-09 14:31   ` Promoting Git developers David Kastrup
2015-03-09 18:32     ` Philip Oakley
2015-03-10  7:45   ` Junio C Hamano
2015-03-10 11:51   ` Promoting Git developers (was: Bashing freelancers) Christian Couder
2015-03-10 17:23     ` Promoting Git developers Junio C Hamano
2015-03-11  1:04       ` Jason St. John
2015-03-11  2:13         ` Duy Nguyen
2015-03-11  4:16           ` Junio C Hamano [this message]
2015-03-12  2:15             ` Duy Nguyen
2015-03-12  4:53               ` Junio C Hamano
2015-03-12  7:45                 ` Fredrik Gustafsson
2015-03-12 18:43                   ` Junio C Hamano
2015-03-11  2:36         ` Junio C Hamano
2015-03-11  7:31           ` Jeff King
2015-03-11  7:38             ` Junio C Hamano
2015-03-11  7:54               ` Jeff King
2015-03-11 21:28                 ` Junio C Hamano
2015-03-11 23:17                   ` Andrew Ardill
2015-03-12 22:31                   ` Jeff King
2015-03-12 22:36                     ` Junio C Hamano
2015-03-12 22:43                       ` Jeff King
2015-03-12  5:05             ` Junio C Hamano
2015-03-12 22:38               ` Jeff King
2015-03-12 22:58                 ` Junio C Hamano
2015-03-15  9:12                   ` Christian Couder
2015-03-11 13:53       ` Christian Couder
2015-03-11 20:42         ` Junio C Hamano
2015-03-15  8:46           ` Christian Couder
2015-03-15 22:18             ` Junio C Hamano
2015-03-15 22:43               ` Randall S. Becker
2015-03-16  9:10                 ` Christian Couder
2015-03-16  9:20                   ` David Kastrup
2015-03-16 17:06                     ` Stefan Beller
2015-03-17 20:08                       ` Christian Couder
2015-03-17 20:16                         ` Junio C Hamano
2015-03-16 23:39               ` David Lang
2015-03-17  5:25                 ` Junio C Hamano
2015-03-17  5:56                   ` David Lang
2015-03-17 20:15               ` Christian Couder
2015-03-17  9:43             ` Thomas Ferris Nicolaisen
2015-03-17 19:51               ` Christian Couder

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:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

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

  git send-email \
    --in-reply-to=xmqqd24g6uf1.fsf@gitster.dls.corp.google.com \
    --to=gitster@pobox.com \
    --cc=christian.couder@gmail.com \
    --cc=dak@gnu.org \
    --cc=git@drmicha.warpmail.net \
    --cc=git@vger.kernel.org \
    --cc=jstjohn@purdue.edu \
    --cc=pclouds@gmail.com \
    --subject='Re: Promoting Git developers' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* 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
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).