Thanks very much Justin for your attention to detail! I've filed BEAM-510, BEAM-513, BEAM-514, BEAM-515 for the website issues you pointed out below, and I've sent PRs for 510 & 513.
Though it may be obvious, I'll remind everyone that these proposed changes to the incubator-beam-site repository do not affect the release artifacts discussed here. Thanks! Dan On Mon, Aug 1, 2016 at 5:17 PM, Justin Mclean <jus...@classsoftware.com> wrote: > Hi, > > +1 (binding) > > I checked: > - file name contains incubating > - signature and hash good > - LICENSE and NOTICE good. (Although not 100% sure why the NOTICE mentions > google twice) > - No binary files in release > - All source code has ASF headers > - Can compile from source > > Just a few minor things I noticed while looking at your web site. You have > a direct link to your last release here [1] rather than a link going via > the Apache mirrors script [5] and this link [2] in Learn menu goes nowhere. > Branding wise I think you are missing a few of the required links [3] > including a link back to the Apache homepage. A feather ASF logo would be a > nice addition as well. [4] > > Thanks, > Justin > > 1 . http://beam.incubator.apache.org/releases/ > 2. http://beam.incubator.apache.org/docs/learn/programming-guide/ > 3. http://www.apache.org/foundation/marks/pmcs.html#navigation > 4. http://www.apache.org/foundation/press/kit/#links > 5. http://www.apache.org/dev/release-download-pages.html#closer > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
