----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62548/#review186296 -----------------------------------------------------------
I like the idea of adding PR submissions for "trivial" improvements. I would suggest adding some guidelines on what that would mean. Our internal team used to have full code reviews for non-trivial changes vs PRs for trivial changes and eventually we've creeped to having only PRs since its that much easier. Some line in the sand would help people understand the divide, and prevent that line from shifting. - Tom Runyon On Sept. 25, 2017, 6:02 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62548/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2017, 6:02 p.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Benjamin Mahler, and Vinod > Kone. > > > Bugs: MESOS-564 > https://issues.apache.org/jira/browse/MESOS-564 > > > Repository: mesos > > > Description > ------- > > Reorganized and updated the contribution guidelines. > > > Diffs > ----- > > docs/home.md f9b35e3e8f9af024a58760c345931d73a83654ff > docs/newbie-guide.md e9f2aaac1bb986120f34b1d006e3a2f5eb2779ff > docs/reopening-reviews.md fe5046830bbdf28fcc2377ffa5792549920afbc8 > docs/reporting-a-bug.md a7e372c1f0d3a34a06244aecb1dfeef7356b8928 > docs/submitting-a-patch.md ffc6e561b8721b8849ef6025c15936ea712d3bfa > site/source/community.html.md f56131fedb935ff695206948e16252c62ae0f36a > > > Diff: https://reviews.apache.org/r/62548/diff/2/ > > > Testing > ------- > > > Thanks, > > Greg Mann > >
