# Help using git I'm review process 9 messages from 2011-07-24 to 2011-07-24. Participants: Mathew Benson, Thomas Adam, Jens Lehmann, Magnus Bäck. Thread: https://gitlist.dev/t/27907 ## Mathew Benson, 2011-07-24 12:12 Subject: Help using git I'm review process Message-ID: <245B4B40-CA0A-43F6-A3F0-85E50D75CB93@gmail.com> URL: https://gitlist.dev/e/245B4B40-CA0A-43F6-A3F0-85E50D75CB93%40gmail.com ``` I'm planning to use git for a work project, which requires tight control of the peer review process. In previous jobs, the peer review was a tedious manual process of creating PDF files, writing comments in spreadsheets, and copying comments to the CM system. I want to use technology to my best advantage. Once a developer has completed all his changes in his development branch, what's the best way to get those files to the reviewers, without requiring the author to stop work? First, I think I should create a tag in the developer branch. Each developer has a local repository, and my review tool writes files directly in the work area. Can they just fetch, checkout a tag (don't know how to do that), commit changes, and push it back to the central repository? Is there a better workflow? ``` ## Thomas Adam, 2011-07-24 12:16 Subject: Re: Help using git I'm review process Message-ID: URL: https://gitlist.dev/e/CA%2B39Oz5OT_RjyGeQdKaFycHjwN6hzy%2BN6P6DDBAtSv3vTYo_4A%40mail.gmail.com In-Reply-To: <245B4B40-CA0A-43F6-A3F0-85E50D75CB93@gmail.com> ``` On 24 July 2011 13:12, Mathew Benson wrote: > I'm planning to use git for a work project, which requires tight control of the peer review process.  In previous jobs, the peer review was a tedious manual process of creating PDF files, writing comments in spreadsheets, and copying comments to the CM system.  I want to use technology to my best advantage. > > Once a developer has completed all his changes in his development branch, what's the best way to get those files to the reviewers, without requiring the author to stop work?  First, I think I should create a tag in the developer branch.  Each developer has a local repository, and my review tool writes files directly in the work area.  Can they just fetch, checkout a tag (don't know how to do that), commit changes, and push it back to the central repository?  Is there a better workflow?-- This is what Gerrit is useful for. -- Thomas Adam ``` ## Mathew Benson, 2011-07-24 12:42 Subject: Re: Help using git I'm review process Message-ID: URL: https://gitlist.dev/e/A5291838-E8F4-47DA-A3E7-E6CBFE7664C9%40gmail.com In-Reply-To: ``` I found that in my research but my IT "department" recommended a tool they're already using. I'll push Gerrit again. Is there an online demo? Sent from my iPad On Jul 24, 2011, at 7:16 AM, Thomas Adam wrote: > On 24 July 2011 13:12, Mathew Benson wrote: >> I'm planning to use git for a work project, which requires tight control of the peer review process. In previous jobs, the peer review was a tedious manual process of creating PDF files, writing comments in spreadsheets, and copying comments to the CM system. I want to use technology to my best advantage. >> >> Once a developer has completed all his changes in his development branch, what's the best way to get those files to the reviewers, without requiring the author to stop work? First, I think I should create a tag in the developer branch. Each developer has a local repository, and my review tool writes files directly in the work area. Can they just fetch, checkout a tag (don't know how to do that), commit changes, and push it back to the central repository? Is there a better workflow?-- > > This is what Gerrit is useful for. > > -- Thomas Adam ``` ## Thomas Adam, 2011-07-24 12:53 Subject: Re: Help using git I'm review process Message-ID: URL: https://gitlist.dev/e/CA%2B39Oz5V3sf0ODqtm%2Bjn3%3Dob9qeSTkVierBcKaCuWXYkjfqobQ%40mail.gmail.com In-Reply-To: ``` On 24 July 2011 13:42, Mathew Benson wrote: > I found that in my research but my IT "department" recommended a tool they're already using.  I'll push Gerrit again.  Is there an online demo? https://review.source.android.com/#/q/status:open,n,z -- Thomas Adam ``` ## Thomas Adam, 2011-07-24 14:02 Subject: Fwd: Help using git I'm review process Message-ID: URL: https://gitlist.dev/e/CA%2B39Oz4-%2BRmX0XqrW9o%2BuxGjTtP3grTUaaZ9FBHdR4GPh21ciw%40mail.gmail.com In-Reply-To: ``` [ Don't mail me off-list; this means don't cull the CC list. ] ---------- Forwarded message ---------- From: Mathew Benson Date: 24 July 2011 14:58 Subject: Re: Help using git I'm review process To: Thomas Adam I know Gerrit integrates with Git.  Does it integrate with Trac? Gerrit does understand c/c++, right? Sent from my iPad On Jul 24, 2011, at 7:53 AM, Thomas Adam wrote: > On 24 July 2011 13:42, Mathew Benson wrote: >> I found that in my research but my IT "department" recommended a tool they're already using.  I'll push Gerrit again.  Is there an online demo? > > https://review.source.android.com/#/q/status:open,n,z > > -- Thomas Adam ``` ## Mathew Benson, 2011-07-24 14:19 Subject: Re: Help using git I'm review process Message-ID: URL: https://gitlist.dev/e/D58B4AAE-55F0-424D-B59E-CE198260F798%40gmail.com In-Reply-To: ``` Sorry. Accidentally pressed reply instead of reply all. Sent from my iPhone On Jul 24, 2011, at 9:02 AM, Thomas Adam wrote: > [ Don't mail me off-list; this means don't cull the CC list. ] > > ---------- Forwarded message ---------- > From: Mathew Benson > Date: 24 July 2011 14:58 > Subject: Re: Help using git I'm review process > To: Thomas Adam > > > I know Gerrit integrates with Git. Does it integrate with Trac? > Gerrit does understand c/c++, right? > > Sent from my iPad > > On Jul 24, 2011, at 7:53 AM, Thomas Adam wrote: > >> On 24 July 2011 13:42, Mathew Benson wrote: >>> I found that in my research but my IT "department" recommended a tool they're already using. I'll push Gerrit again. Is there an online demo? >> >> https://review.source.android.com/#/q/status:open,n,z >> >> -- Thomas Adam ``` ## Jens Lehmann, 2011-07-24 14:37 Subject: Re: Help using git I'm review process Message-ID: <4E2C2E15.8010602@web.de> URL: https://gitlist.dev/e/4E2C2E15.8010602%40web.de In-Reply-To: ``` Am 24.07.2011 14:16, schrieb Thomas Adam: > On 24 July 2011 13:12, Mathew Benson wrote: >> I'm planning to use git for a work project, which requires tight control of the peer review process. In previous jobs, the peer review was a tedious manual process of creating PDF files, writing comments in spreadsheets, and copying comments to the CM system. I want to use technology to my best advantage. >> >> Once a developer has completed all his changes in his development branch, what's the best way to get those files to the reviewers, without requiring the author to stop work? First, I think I should create a tag in the developer branch. Each developer has a local repository, and my review tool writes files directly in the work area. Can they just fetch, checkout a tag (don't know how to do that), commit changes, and push it back to the central repository? Is there a better workflow?-- > > This is what Gerrit is useful for. Yes, Gerrit is a very sophisticated way to do that. But you can also achieve a review process by just using git and email too: Have each developer do each topic of his work on a separate feature branch and send merge requests (e.g. per email) to the reviewer when he is done. If the reviewer approves the changes, he merges that branch (and deletes the remote topic branch, as that topic is now finished and part of the history). If not, he requests improvements from the developer who updates that branch and sends another review request when he is done. We use this approach successfully at my dayjob. ``` ## Mathew Benson, 2011-07-24 15:32 Subject: Re: Help using git I'm review process Message-ID: <01E430F0-0FDC-4FBD-9973-0E5B437FACEC@gmail.com> URL: https://gitlist.dev/e/01E430F0-0FDC-4FBD-9973-0E5B437FACEC%40gmail.com In-Reply-To: <4E2C2E15.8010602@web.de> ``` Great. That confirms what I thought. Thanks. On Jul 24, 2011, at 9:37 AM, Jens Lehmann wrote: > Am 24.07.2011 14:16, schrieb Thomas Adam: >> On 24 July 2011 13:12, Mathew Benson wrote: >>> I'm planning to use git for a work project, which requires tight control of the peer review process. In previous jobs, the peer review was a tedious manual process of creating PDF files, writing comments in spreadsheets, and copying comments to the CM system. I want to use technology to my best advantage. >>> >>> Once a developer has completed all his changes in his development branch, what's the best way to get those files to the reviewers, without requiring the author to stop work? First, I think I should create a tag in the developer branch. Each developer has a local repository, and my review tool writes files directly in the work area. Can they just fetch, checkout a tag (don't know how to do that), commit changes, and push it back to the central repository? Is there a better workflow?-- >> >> This is what Gerrit is useful for. > > Yes, Gerrit is a very sophisticated way to do that. > > But you can also achieve a review process by just using git and email > too: Have each developer do each topic of his work on a separate feature > branch and send merge requests (e.g. per email) to the reviewer when he > is done. If the reviewer approves the changes, he merges that branch > (and deletes the remote topic branch, as that topic is now finished and > part of the history). If not, he requests improvements from the developer > who updates that branch and sends another review request when he is done. > We use this approach successfully at my dayjob. ``` ## Magnus Bäck, 2011-07-24 15:40 Subject: Re: Fwd: Help using git I'm review process Message-ID: <20110724154014.GA28962@jpl.local> URL: https://gitlist.dev/e/20110724154014.GA28962%40jpl.local In-Reply-To: ``` On Sunday, July 24, 2011 at 14:58, Mathew Benson wrote: > I know Gerrit integrates with Git. Does it integrate with Trac? That depends on what you mean by "Trac integration". Can you use Gerrit and Trac together? Yes. Can Gerrit speak directly to a Trac server and do stuff based on e.g. review events? No. Does Gerrit have interfaces that would allow you to create glue that'll connect the two tools? Yes, it might be possible depending on your answer to my first question. > Gerrit does understand c/c++, right? Gerrit does support syntax highlighting of several languages. Apart from that it doesn't care about what language you write your code in. -- Magnus Bäck Opinions are my own and do not necessarily SW Configuration Manager represent the ones of my employer, etc. Sony Ericsson ```