On March 30 2012 at 21:25, Platonides wrote:
<snip>
How would you script that if you don't have the
files? (as they are
pending a merge)
Could we have a branch which included all non-abandoned patches? Or
maybe all patches whose total feedback is not negative.
One just have to fetch the non-merged change from Gerrit. Git magic is:
# Fetch change 1234
$ git fetch gerrit refs/changes/34/1234
# Switch to that branch:
$ git checkout FETCH_HEAD
Remember that "refs/changes/34/1234" is just a pointer to some commit
object exactly like "master". So you can switch to it, run your tests
and submit their results.
<snip>
We should have
less fixme nowadays since we have adopted a pre merge
review, <snip>
Can someone measure at CodeReview the number of revisions which went to
fixme after having been on ok? gerrit system allowing pre-review doesn't
help with the 'false review rate'.
There *will* be bugs which get merged into the main repo. Not every
master status will be perfectly stable, as we wish it were.
Ability to mark the patchsets as fixme is a good tool. If we had a list
of follow-ups in gerrit, that would also be useful.
We already have the fixme feature. That is done by down voting a
patchset in the codereview field (the infamous : "I would prefer that
you didn't submit this").
The follow up we have been abusing is also build in since you usually
just send a second patch. A follow up to a previous merge is either a
new feature or a bug fix, it can still reference the change number but I
am not sure there is any value in doing so.
Whenever someone would need to use followup, he should probably use a
branch instead. Branch are cheap, use them :-]
--
Antoine "hashar" Musso