----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65767/#review198321 -----------------------------------------------------------
Ship it! I looked at the code with a friend who knows Ruby. In general, it looks good. The only comment we had is that the "archived_source_release_link" method isn't being used. Based on the comment, we understood that the intention is to use it in the future, when 1.3.3 becomes a "past" release, since 1.3.0-1.3.2 had URLs inconsistent with 1.3.3 (and future releases), and therefore the method doesn't apply. Is that right? - Eyal Allweil On Feb. 23, 2018, 7:13 a.m., Matthew Hayes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65767/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2018, 7:13 a.m.) > > > Review request for DataFu. > > > Repository: datafu > > > Description > ------- > > Updated according to https://issues.apache.org/jira/browse/DATAFU-137 > > I wrote helper methods to format the links for the releases. Because older > releases used different and inconsistent extensions I didn't write a method > for these. > > > Diffs > ----- > > site/config.rb 745a3f8 > site/source/docs/download.html.markdown.erb 0a400f5 > > > Diff: https://reviews.apache.org/r/65767/diff/1/ > > > Testing > ------- > > > Thanks, > > Matthew Hayes > >
