> -----Original Message-----
> From: Ben Reser [mailto:ben_at_reser.org]
> Sent: woensdag 9 april 2014 20:07
> To: Branko Čibej; Subversion Development
> Subject: Re: remote-only-status branch ready for review
>
> On 4/9/14, 9:51 AM, Branko Čibej wrote:
> > I've finished the work on the remote-only-status branch. I'd like to ask
> > for a review of the changes before I merge it to trunk. I'm currently
> > running a full round of tests on the branch, which I've sync'd with
> > trunk up to r1585988; results are looking good for now.
> >
> > If the tests pass, and there are no objections, I'll merge to trunk
> > early next week.
> >
> > svn diff ^/subversion/trunk ^/subversion/branches/remote-only-status
>
> Looks good to me. I noticed the whitespace things that Bert mentioned on
> IRC.
> Not going to point them out since you seem to already be aware of them.
+1
I agree... Looks good.
Personally I don't see remaining reasons that would require to wait until next week, but that is your call.
If there are remaining issues they are small enough to handle them on trunk.
(And the resulting total branch-patch looks small enough to review in the final commit for others)
Bert
Received on 2014-04-09 20:19:06 CEST