> On Sept. 4, 2015, 3:45 p.m., Michael Park wrote: > > docs/configuration.md, lines 1502-1510 > > <https://reviews.apache.org/r/37814/diff/3/?file=1060676#file1060676line1502> > > > > Hm, looks like the closing tag, `</tr>` is missing here...? It also > > looks like it's missing in all of the other tables in this file. Is this > > valid HTML?
According to this: http://www.w3.org/TR/html5/syntax.html#optional-tags it is legal, but it's probably a bit sloppy :-) I went ahead and added the closing tags throughout the document to make things consistent. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37814/#review97761 ----------------------------------------------------------- On Sept. 4, 2015, 4:14 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37814/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2015, 4:14 p.m.) > > > Review request for mesos, Alexander Rojas, haosdent huang, and Michael Park. > > > Bugs: MESOS-2466 > https://issues.apache.org/jira/browse/MESOS-2466 > > > Repository: mesos > > > Description > ------- > > Added documentation for libprocess environment variables > > > Diffs > ----- > > docs/configuration.md 2b23d48c8841e43e6e2776dfe6bfa7c022a941a7 > > Diff: https://reviews.apache.org/r/37814/diff/ > > > Testing > ------- > > Built the updated website using the docker site builder > (https://github.com/mesosphere/mesos-website-container). > > > Thanks, > > Greg Mann > >
