Fwd: git-reviewed: linking commits to review discussion in git

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello All,

We have linked peer review discussions on
git@xxxxxxxxxxxxxxx to their respective commits within the main
git.git repository. You can view the linked reviews from 2012
until present in the GitHub repo at:
https://github.com/mmukadam/git/tree/review

If you want to search for the reviews related to a commit, you can
type in the full hash code here:
http://users.encs.concordia.ca/~m_mukada/git-review-tracker.html

or append the commit hash to the following url:
https://github.com/mmukadam/git/tree/review/<commit_hash>

Note, if you get a '404 page not found' from GitHub it means that we
couldn't find a review to link the commit to.

It's a bit hard to view the reviews on GitHub because the 'review' branch was
designed to be viewed on the command line using a set of simple scripts that
sit on top of the standard git commands.

For example:
to view all reviews Hamano is involved in you can do:
git reviewed --log-reviewer hamano

to view the reviews of particular commit:
git reviewed --log 1a1832eb085e5bca198735e5d0e766a3cb61b8fc

You can also create new reviews on a commit and mail them as well as
import reviews from email.

Note the command is git-reivewED because git-review is associated with Gerrit.

The tool is beta, but it just creates a detached review branch, so it
won't affect master. To remove it all you do is delete the scripts and
the 'review' branch. More info available here:

The online man-page is here:
http://users.encs.concordia.ca/~m_mukada/git-review.html

In order to install git-review, you need to clone the repository:
https://github.com/mmukadam/git-reviewed.git

The online tutorial is available here:
http://users.encs.concordia.ca/~m_mukada/git-review-tutorial.html


Sample output:

Let's say you want to see the reviews related to commit:
---------------------------------------------------------------------------------
git reviewed --log 9ce415d972a04df4bfceaad0fab8eaea9a66997e

Review: 74d203accc341993dbed507fceceb2841670ce37
Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Author:     Johannes Sixt <j6t@xxxxxxxx>
AuthorDate: Fri Jun 7 22:53:27 2013 +0200
Reviewer:   Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx>
ReviewDate: Thu Feb 13 11:35:13 2014 -0800


    Re: [PATCH] tests: turn on network daemon tests by default

Review: e80ebaa1fdd132a13bfbc6f87cdd4ecec7f2b715
Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Author:     Johannes Sixt <j6t@xxxxxxxx>
AuthorDate: Fri Jun 7 22:53:27 2013 +0200
Reviewer:   Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx>
ReviewDate: Wed Feb 12 14:34:52 2014 -0800


    Re: [PATCH] tests: turn on network daemon tests by default

Review: 8da03766afcd60e432ffdd2d6f549e2ed02a4bc9
Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Author:     Johannes Sixt <j6t@xxxxxxxx>
AuthorDate: Fri Jun 7 22:53:27 2013 +0200
Reviewer:   Jeff King peff@xxxxxxxx <peff@xxxxxxxx>
ReviewDate: Mon Feb 10 16:29:37 2014 -0500


    [PATCH] tests: turn on network daemon tests by default



To show the email review discussion for a single review:
---------------------------------------------------------------------------------

git reviewed --view 74d203accc341993dbed507fceceb2841670ce37

Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Reviewer: Junio C Hamano <gitster@xxxxxxxxx>
Reviewer Date: Thu, 13 Feb 2014 11:35:13 -0800
---------------------------------------------------------------
>From : Junio C Hamano <gitster@xxxxxxxxx>
Message-Id : <xmqqfvnmlsb2.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
Date : Thu, 13 Feb 2014 11:35:13 -0800
Subject: Re: [PATCH] tests: turn on network daemon tests by default
References: <20140210191521.GA3112@xxxxxxxxxxxxxxxxxxxxx>
<20140210212931.GA16154@xxxxxxxxxxxxxxxxxxxxx>
<xmqqa9dxpgw9.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
<20140211200445.GA27946@xxxxxxxxxxxxxxxxxxxxx>
<xmqqzjlxnqvw.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
<20140212214753.GA6799@xxxxxxxxxxxxxxxxxxxxx>
<xmqq38jom037.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
In-Reply-To: <xmqq38jom037.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> (Junio C.
Hamano's message of "Wed, 12 Feb 2014 14:34:52 -0800")
Cc: Duy Nguyen <pclouds@xxxxxxxxx>, Git Mailing List
<git@xxxxxxxxxxxxxxx>, "Shawn O. Pearce" <spearce@xxxxxxxxxxx>
commit: 9ce415d972a04df4bfceaad0fab8eaea9a66997e

Junio C Hamano <gitster@xxxxxxxxx> writes:

> Jeff King <peff@xxxxxxxx> writes:
>
>>   test_normalize_tristate GIT_TEST_DAEMON
>
> Heh, great minds think alike.  This is what I am playing with,
> without committing (because I do like your "ask config if this is a
> kind of various boolean 'false' representations, which I haven't
> managed to add to it).

[snip]

---
 t/lib-git-daemon.sh     |  8 ++++---
 t/lib-httpd.sh          | 22 +++++++++----------
 t/test-lib-functions.sh | 58 +++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 74 insertions(+), 14 deletions(-)

diff --git a/t/lib-git-daemon.sh b/t/lib-git-daemon.sh
index 394b06b..615bf5d 100644
--- a/t/lib-git-daemon.sh
+++ b/t/lib-git-daemon.sh
@@ -16,9 +16,10 @@
 #      stop_git_daemon
 #      test_done

-if test -z "$GIT_TEST_GIT_DAEMON"
+test_tristate GIT_TEST_GIT_DAEMON
+if test "$GIT_TEST_GIT_DAEMON" = false
 then
-       skip_all="git-daemon testing disabled (define
GIT_TEST_GIT_DAEMON to enable)"
+       skip_all="git-daemon testing disabled (unset
GIT_TEST_GIT_DAEMON to enable)"
        test_done
 fi

[snip]

 # The following mingw_* functions obey POSIX shell syntax, but are actually
 # bash scripts, and are meant to be used only with bash on Windows.

--
1.9.0-rc3-260-g4cf525c


To see all the reviews anyone is involved in:
-------------------------------------------------------------------

git reviewed --log reviewer hamano

Review: ecb6f98955700169ed6f88d929babd7d0f770116
Commit Reviewed: 9c8ce7397bac108f83d77dfd96786edb28937511
Author:     Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx>
AuthorDate: Thu Feb 13 11:35:08 2014 +0100
Reviewer:   Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx>
ReviewDate: Thu Feb 13 13:25:23 2014 -0800


    Re: [PATCH] release notes: typo fixes

Review: 74d203accc341993dbed507fceceb2841670ce37
Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Author:     Johannes Sixt <j6t@xxxxxxxx>
AuthorDate: Fri Jun 7 22:53:27 2013 +0200
Reviewer:   Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx>
ReviewDate: Thu Feb 13 11:35:13 2014 -0800


    Re: [PATCH] tests: turn on network daemon tests by default

Review: e80ebaa1fdd132a13bfbc6f87cdd4ecec7f2b715
Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e
Author:     Johannes Sixt <j6t@xxxxxxxx>
AuthorDate: Fri Jun 7 22:53:27 2013 +0200
Reviewer:   Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx>
ReviewDate: Wed Feb 12 14:34:52 2014 -0800


    Re: [PATCH] tests: turn on network daemon tests by default



Please let us know what you think about this tool as it will help us
in improving it. If you'd like us to extract the reviews related to a
different repository, please let us know.

This is part of my Msc thesis, so any feedback is appreciated.
We thank you for you time.

Cheers,
Murtuza
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]