[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
[tycho-dev] FW: Gerrit review process
|
-----Original Message-----
From: Oberlies, Tobias
Sent: Montag, 23. April 2012 16:23
To: 'Igor Fedorenko'
Cc: Sievers, Jan
Subject: Gerrit review process
I would propose that we do the rebases ourselves, e.g.
* when starting the review of a change (if there have been relevant concurrent changes), and
* directly before committing it.
The contributors can't know when we are going to review their changes, i.e. when it makes sense to do a rebase, and therefore probably would do it too often. And with our policy to require a Change-Id, a rebase is usually no effort at all.
The rebase will change the commiter, but leave the author unchanged. Only the latter is relevant for the IP log.
Regards
Tobias
> -----Original Message-----
> From: Igor Fedorenko [mailto:igor@xxxxxxxxxxxxxx]
> Sent: Montag, 23. April 2012 16:07
> To: Oberlies, Tobias
> Cc: Sievers, Jan
> Subject: Re: [Bug 371620] Enable Gerrit for tycho
>
> What do we do for changes submitted by external developers, like
> https://git.eclipse.org/r/#/c/5642/ ? Do we ask them to do rebase&push
> or do we do this ourselves? In the latter case, will the commit still be
> attributed to the original author?
>
> --
> Regards,
> Igor
>
> On 12-04-23 5:49 AM, Oberlies, Tobias wrote:
> > ... and for the master branch, it is refs/for/master
> >
> > Just to be sure: Igor, are you aware that your change
> https://git.eclipse.org/r/#/c/5567/ is not yet submitted (i.e. merged to
> the master branch)? To get it in, you need to
> > * rebase it to origin/master
> > * push it to refs/for/master again
> > * vote on the change
> > * submit it
> >
> > Best regards
> > Tobias
> >