[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: remote-only-status branch ready for review

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 9 Apr 2014 20:18:24 +0200

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

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)

Received on 2014-04-09 20:19:06 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.