> On April 5, 2014, 11:49 p.m., Bill Farner wrote: > > Generally for markdown reviews, we like having the diff pushed to a branch > > on a repo fork on github. This makes life easy for the reviewer to see > > exactly how the finished product will look. > > > > Review meta-comment: please adjust the summary to reflect what's being > > done. in the commit history, i'd rather see something like "Convert docs > > tables from HTML to pure markdown" as opposed to "Consider using...". > > Dan Norris wrote: > Re: meta-comment: Does that mean that I should update the ticket title in > Jira as well? 99% of the time that's where I get the summary header from. > > Bill Farner wrote: > I'll leave that up to you, i just prefer that the commit log reflects > actions rather than problems. Tickets, on the other hand, will often be > summarized with a problem.
That's a good guideline to follow, thanks for the clarification! I already updated both the ticket and the title of this review to describe what was done. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20027/#review39640 ----------------------------------------------------------- On April 9, 2014, 2:31 a.m., Dan Norris wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20027/ > ----------------------------------------------------------- > > (Updated April 9, 2014, 2:31 a.m.) > > > Review request for Aurora, Jake Farrell and Bill Farner. > > > Bugs: AURORA-234 > https://issues.apache.org/jira/browse/AURORA-234 > > > Repository: aurora > > > Description > ------- > > AURORA-234: Consider using markdown-style tables instead of HTML > > > Diffs > ----- > > docs/configurationreference.md 9d5c340025d2d14be37ef7d1a67f186c8c1792db > docs/deploying-aurora-scheduler.md 887bf1691068037be5f9adf4ddb9e999ec1a899b > docs/hooks.md 77fb95598b9f86c36979089cd4aa798044329367 > > Diff: https://reviews.apache.org/r/20027/diff/ > > > Testing > ------- > > Generated the Aurora site using Middleman. Everything renders correctly, > though I've updated the Jira ticket with a comparison between the two > rendered tables. > > Commit on Github: > https://github.com/protochron/incubator-aurora/commit/558a5a6a2f15a82e51ef7bf6d17288f67ff11a97 > > > Thanks, > > Dan Norris > >