----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31327/#review73994 -----------------------------------------------------------
Ship it! Looks great! Just a couple nits. docs/configuration.md <https://reviews.apache.org/r/31327/#comment120431> Where are the matching `<p>`s for these `</p>`s? You can omit the end tag, but not the begin tag. Also, perhaps because of this, the NOTE doesn't show up on a new line at https://github.com/nqn/mesos/blob/niklas/updated-configuration/docs/configuration.md docs/configuration.md <https://reviews.apache.org/r/31327/#comment120432> Did you mean `<p/>`? - Adam B On Feb. 24, 2015, 4:54 p.m., Niklas Nielsen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31327/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2015, 4:54 p.m.) > > > Review request for mesos, Ben Mahler, Kapil Arya, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Added new flags and removed checkpoint slave flag > > > Diffs > ----- > > docs/configuration.md 47d8ccb6a1f318b153a3509be10b8dac9a24130c > > Diff: https://reviews.apache.org/r/31327/diff/ > > > Testing > ------- > > > Thanks, > > Niklas Nielsen > >
