Thanks for taking a look, Christian! I'll take a look at the issues you found, and get back with an update.
Tim -----Original Message----- From: Christian Grobmeier [mailto:[email protected]] Sent: Friday, March 13, 2015 5:16 PM To: [email protected] Subject: Re: [VOTE] Ripple release 0.9.28 I found the following issues: NOTICE -> 2012 :) pkg/hosted/ripple.js appears to be empty. Is that correct? pkg/hosted do not have license headers. It looks like this would generated code, which is uploaded to somewhere? In Java-terms it would be similar to a binary artifact, which also do not have headers. This might come up as an issue. At Apache we are releasing source code first, everything else is just nice. The best and easiest thing would be to just add the header (automatically) to that files. Are there any options? /assets/server/images/NOTICE: its in a folder where only the logo remains. Is the location intended? I see a lot of images in /pkg/hosted/images, but no NOTICE there Maybe the included message should just go to the global NOTICE file? Thanks, i feel we are close :) Christian -- Christian Grobmeier http://www.grobmeier.de http://www.timeandbill.de On Tue, Mar 10, 2015, at 15:14, Tim Barham wrote: > Please review and vote on the release of Ripple 0.9.28. > > The package you are voting on is available for review at > http://bit.ly/1FZ8meZ. It was published from its corresponding git tag: > incubator-ripple: 0.9.28 (1d95fed542) > > Since this will be an official Apache release of Ripple (our first!), we > must be particularly careful that it complies with all Apache guidelines > for an incubator release. As such, before voting +1, please refer to and > verify compliance with the checklist at > http://incubator.apache.org/guides/releasemanagement.html#check-list. > > If anyone has concerns that we don't meet any of these requirements, > please don't hesitate to raise them here so we can discuss and make > changes if necessary. > > If you do give a +1 vote, please include what steps you took in order to > be confident in the release. > > Please also note from Ross's recent email: > > > What we need is three +1 "binding" votes, in reality that means three IPMC > > members. Once a project graduates it means three project management > > committee > > members. However, as a mentor (therefore having a binding vote) I look to > > the > > project participants to indicate their preference and (assuming no blocking > > issues on an IP check) I'll always vote in support of the communities non- > > binding votes. > > So please, even though your vote may not be binding, take some time to > review the release and vote! > > Upon a successful vote, we will arrange for the archive to be uploaded to > dist/incubator/ and publish it to NPM. > > Thanks, and looking forward to our first official Ripple release! > > Tim
