> On Feb. 6, 2018, 7:12 p.m., Andrew Schwartzmeyer wrote: > > Nit: Fix summary to be in past tense, and fix description to not be a copy > > of the summary. > > Jeff Coffler wrote: > This was a one-line commit, but I went ahead and added a second line. I'm > not sure why you said that the description shouldn't be a copy of the > summary, it wasn't. Tooling problem on your side, or just a mistake? > > Andrew Schwartzmeyer wrote: > When you post a review with just a summary, the scripts copy the summary > into the description. This unfortunately carries through to the other side > when the review is applied, and then you have a weird commit. The > (work-around?) is to not post one-line commit messages.
That's odd. I think you mentioned this before, but I see one-line commits on Mesos, like this: ``` commit 2ef5f4a Author: Gilbert Song <songzihao1...@gmail.com> Date: Thu Feb 8 08:45:42 2018 Added an image 1.3-1.5_v1_json_state_query_latency.png to docs/images. commit f7dbd29 Author: Michael Park <mp...@apache.org> Date: Wed Feb 7 20:28:42 2018 Updated ReviewBot to use Ubuntu 16.04. ``` Am I doing something wrong. Do I need something "funny" in the description (like a "." or something) if it's one line? - Jeff ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65508/#review196911 ----------------------------------------------------------- On Feb. 5, 2018, 6:12 p.m., Jeff Coffler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65508/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2018, 6:12 p.m.) > > > Review request for mesos, Andrew Schwartzmeyer and John Kordich. > > > Bugs: MESOS-6715 > https://issues.apache.org/jira/browse/MESOS-6715 > > > Repository: mesos > > > Description > ------- > > Allow curl program to be properly executed on Windows platform. > > > Diffs > ----- > > src/uri/fetchers/curl.cpp f34daf2bc1f3fef74548e04ad332e3a627636bfd > > > Diff: https://reviews.apache.org/r/65508/diff/1/ > > > Testing > ------- > > Full build, unit tests on both Linux and Windows platform. > > > Thanks, > > Jeff Coffler > >