> On May 5, 2016, 12:26 p.m., Joerg Schad wrote: > > docs/app-framework-development-guide.md, line 10 > > <https://reviews.apache.org/r/47008/diff/1/?file=1372970#file1372970line10> > > > > 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...
Good point, Jörg; let's bring this up on the devlist. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47008/#review131814 ----------------------------------------------------------- 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 > >
