git-review: linking commits to review discussion in git

From: Murtuza Mukadam <murtuza(dot)i(dot)mukadam(at)gmail(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: git-review: linking commits to review discussion in git
Date: 2014-01-27 21:36:34
Message-ID: CANx4bCAaBiYZxGb_KWCJXHvbMHGahmf+8bwsm0=oVWrprO-4KQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello All,
We have linked peer review discussions on
'pgsql-hackers' to their respective commits within the main
postgresql.git repository. You can view the linked reviews from 2012
until present in the GitHub repo at
https://github.com/mmukadam/postgres/tree/review

If you want to work with these reviews locally, you can use our
git-review tool. It allows you to create reviews and attach them to
commits in git. We didn't modify git, instead we added some scripts
that use standard git commands. git-review is beta, but since it only
adds a detached 'review' branch and modifies the contents of this
branch, it has minimal impact and can easily be removed by deleting
the 'review' branch and scripts.

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-review.git

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

The clone of postgresql.git with linked review discussion is here (new
review discussion are linked nightly)
https://github.com/mmukadam/postgres

This work is part of my Master's thesis. If you'd like us to change
the tool to better suit your review process, have another git repo
you'd like us to link commits with review discussion, or have other
feedback, please let us know.

Cheers,
Murtuza

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2014-01-27 21:37:10 Re: Add min and max execute statement time in pg_stat_statement
Previous Message Simon Riggs 2014-01-27 21:16:32 Re: Freezing without write I/O