Re: Review Request 41732: Slight documentation overhaul

2015-12-28 Thread Stephan Erb
> On Dec. 27, 2015, 11:20 p.m., Stephan Erb wrote: > > Looks like you have pushed the previous version without the review changes. > > Bill Farner wrote: > Oy, sorry about that. I yanked it from your branch (since i can't pull > the patch with binaries) and neglected to check if it

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/ --- (Updated Dec. 27, 2015, 12:57 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/#review111934 --- Thanks! This was sorely needed. Only a few very minor issues.

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/ --- (Updated Dec. 27, 2015, 10:46 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Bill Farner
> On Dec. 27, 2015, 2:20 p.m., Stephan Erb wrote: > > Looks like you have pushed the previous version without the review changes. Oy, sorry about that. I yanked it from your branch (since i can't pull the patch with binaries) and neglected to check if it matched the review patch. I don't

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/#review111941 --- This patch does not apply cleanly against master (1b43126), do you

Re: Review Request 41732: Slight documentation overhaul

2015-12-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/#review111944 --- Looks like you have pushed the previous version without the review

Re: Review Request 41732: Slight documentation overhaul

2015-12-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41732/#review111913 --- Ship it! Master (1ae77d5) is green with this patch.