Re: [CSV] release review

2014-05-05 Thread Benedikt Ritter
Hello Bernd, many thanks for your detailed review. Here are my comments: 2014-05-04 21:11 GMT+02:00 Bernd Eckenfels e...@zusammenkunft.net: Hello, I did some chekcs on the CSV trunk cause you mentioned a release is coming up (and I feel its a bad feeling to get no external reviews for

Re: [CSV] release review

2014-05-05 Thread Benedikt Ritter
Hello Bernd, 2014-05-04 23:57 GMT+02:00 Bernd Eckenfels e...@zusammenkunft.net: Hello, I noticed while building, that there is a warning of the taglist plugin regarding deprecated config option. When I use the following it will use the new configuration mehtod and does some more tags

[CSV] release review

2014-05-04 Thread Bernd Eckenfels
Hello, I did some chekcs on the CSV trunk cause you mentioned a release is coming up (and I feel its a bad feeling to get no external reviews for releases). Here are my findings: - compiling with Java 8 and maven 3.2.1 on Windows works, but the site goal fails (findbugs porblem) - site:

Re: [CSV] release review

2014-05-04 Thread Bernd Eckenfels
Hello, I noticed while building, that there is a warning of the taglist plugin regarding deprecated config option. When I use the following it will use the new configuration mehtod and does some more tags (separated in two sections): plugin groupIdorg.codehaus.mojo/groupId