----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32998/#review80786 -----------------------------------------------------------
docs/committing.md <https://reviews.apache.org/r/32998/#comment130904> s/committers/Apache Mesos Committers/? s/change/changes/? s/one/a committer/ docs/committing.md <https://reviews.apache.org/r/32998/#comment130905> Very long line. Do we have a style guide for markdown somewhere? I would think that 80 cols would work :) docs/committing.md <https://reviews.apache.org/r/32998/#comment130906> You chose to break the line here, so would it make sense to do a scan and wrap? docs/committing.md <https://reviews.apache.org/r/32998/#comment130907> It would be great to have a todo or somehow refer to a link with a doc specifying our supported OS's, compilers etc. :) docs/effective-code-reviewing.md <https://reviews.apache.org/r/32998/#comment130911> s/, this is not meant to be a sparring match!/./ docs/effective-code-reviewing.md <https://reviews.apache.org/r/32998/#comment130913> s/**//g - Niklas Nielsen On April 8, 2015, 5:30 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32998/ > ----------------------------------------------------------- > > (Updated April 8, 2015, 5:30 p.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Jie Yu, Niklas Nielsen, > and Vinod Kone. > > > Bugs: MESOS-2581 > https://issues.apache.org/jira/browse/MESOS-2581 > > > Repository: mesos > > > Description > ------- > > "Committer's Guide" was too generic. This names the documents after "what" > the reader is looking for: doing effective reviews, and how to commit changes > (for committers only). > > > Diffs > ----- > > docs/committers-guide.md c016ee9cb3290d7788ed258904547b59bbea4f11 > docs/committing.md PRE-CREATION > docs/effective-code-reviewing.md PRE-CREATION > docs/home.md 6ab61f85aa7d62e0f19267b886dffb4e0aa826ea > > Diff: https://reviews.apache.org/r/32998/diff/ > > > Testing > ------- > > N/A > > > Thanks, > > Ben Mahler > >
