----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47008/#review131814 -----------------------------------------------------------
Ship it! docs/app-framework-development-guide.md (line 10) <https://reviews.apache.org/r/47008/#comment195845> I am not sure we have an official 80 char limit, it is ok for me to change it to make it more readable but there is afaik no guideline for this... - Joerg Schad On May 5, 2016, 12:19 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47008/ > ----------------------------------------------------------- > > (Updated May 5, 2016, 12:19 p.m.) > > > Review request for mesos and Joerg Schad. > > > Repository: mesos > > > Description > ------- > > This change includes: > * Wrap text on 80 chars for consistency and readability; > * Wrap argument list on '(' for consistency with "scheduler.hpp"; > * Enclose protobuf message names in backticks. > > > Diffs > ----- > > docs/app-framework-development-guide.md > c5badd00121e7d1752dd2a4b471f97b80ef07926 > > Diff: https://reviews.apache.org/r/47008/diff/ > > > Testing > ------- > > Visual inspection in MacDown 0.4.3 > > > Thanks, > > Alexander Rukletsov > >