On Feb 20, 2014 10:11 PM, "Gary Gregory" <garydgreg...@gmail.com> wrote: > > I would also like to see reports for FindBugs and PMD.
Will do. > Also: use > commons-parent 33 to pick up the new skin. > Has been deployed and will of course be part of the next RC site. Matt > Gary > > > On Thu, Feb 20, 2014 at 12:33 PM, Mark Struberg <strub...@yahoo.de> wrote: > > > +1 > > > > >The poms don't include any groupId definitions. > > > > Not needed and sometimes even a source of problems. If you do not need to > > write something than adding it only raises the chance it gets wrong. > > > > > > > > > - The jar artifacts contain LICENSE and NOTICE without the .txt > > extension. > > That's perfectly fine. All my ASF projects look that way. > > > > > > > - There is a bunch of checkstyle errors. For an initial release it > > > > > would be good to start with a clean code base. > > I agree, but that's not a show stopper. Those are only warnings. > > > > > > Regarding the mvn clean discussion: We could 'fix' this by using the > > maven-it-plugin. But that makes many things much more complicated to test > > and debug. Thus this is imo nothing we should address soon and all is fine > > imo. > > > > > > LieGrue, > > strub > > > > > > On Wednesday, 12 February 2014, 23:50, sebb <seb...@gmail.com> wrote: > > > > On 12 February 2014 02:20, Matt Benson <mben...@apache.org> wrote: > > >> I would like to make the inaugural release of the [weaver] component. > > >> > > >> Apache Commons Weaver 1.0 RC1 is available for review at: > > >> https://dist.apache.org/repos/dist/dev/commons/weaver/ (r4368). > > >> > > >> Maven artifacts are at: > > >> > > https://repository.apache.org/content/repositories/orgapachecommons-1007/. > > >> > > >> Tested with Oracle JDKs 6 and 7. > > >> > > >> The Subversion tag is: > > >> > > http://svn.apache.org/repos/asf/commons/proper/weaver/tags/1.0_RC1/(r1567477) > > . > > >> > > >> Site: > > >> http://people.apache.org/~mbenson/commons-weaver-1.0-rc1/index.html > > >> > > >> RAT Report: > > >> > > http://people.apache.org/~mbenson/commons-weaver-1.0-rc1/rat-report.html > > >> > > > > > >I've never seen the AL header in release notes before. > > >Not sure that's necessary (and it makes the notes harder to read). > > > > > >The poms don't include any groupId definitions. > > >Although this will default from the parent, I think it is better to > > >specify the group id. > > >Otherwise it is not clear whether the omission is accidental or > > deliberate. > > >Also if the parent group Id ever changes (or perhaps is removed) the > > >component groupId will change unless the groupId is added at that > > >point. > > > > > > > > >> Keys: https://dist.apache.org/repos/dist/release/commons/KEYS > > >> > > >> Please review the release candidate and vote. > > >> This vote will close no sooner that 72 hours from now, i.e. after > > 0300UTC > > >> 15-February 2014 > > >> > > >> [ ] +1 Release these artifacts > > >> [ ] +0 OK, but... > > >> [ ] -0 OK, but really should fix... > > >> [ ] -1 I oppose this release because... > > >> > > >> Thanks! > > > > > >--------------------------------------------------------------------- > > >To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > >For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > > > > > > > > > > > > > > -- > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org > Java Persistence with Hibernate, Second Edition< http://www.manning.com/bauer3/> > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> > Spring Batch in Action <http://www.manning.com/templier/> > Blog: http://garygregory.wordpress.com > Home: http://garygregory.com/ > Tweet! http://twitter.com/GaryGregory
