Hi Simo, 2012/7/16 Simone Tripodi <[email protected]>: > Impressive feedbacks Gary, thanks a lot! > >> -1 with "mvn clear site" for >> https://repository.apache.org/content/repositories/orgapachecommons-054/commons-dbutils/commons-dbutils/1.5/commons-dbutils-1.5-src.zip >> : >> >> [ERROR] Failed to execute goal >> org.apache.maven.plugins:maven-site-plugin:3.0:site (default-site) on >> project commons-dbutils: Error during page generation: Erro >> r rendering Maven report: Could not find resource >> 'C:\temp\commons-dbutils-1.5-src/pmd-ruleset.xml'. -> [Help 1] >> >> My setup: >> >> Apache Maven 3.0.4 (r1232337; 2012-01-17 03:44:56-0500) >> Maven home: C:\Java\apache-maven-3.0.4\bin\.. >> Java version: 1.6.0_31, vendor: Sun Microsystems Inc. >> Java home: C:\Program Files\Java\jdk1.6.0_31\jre >> Default locale: en_US, platform encoding: Cp1252 >> OS name: "windows 7", version: "6.1", arch: "amd64", family: "windows" > > has to be fixed - no doubts :) > >> Minor: >> >> PMD says: Avoid unused private methods. >> > > do you have any hint on how to have them fixed? I mean (IIUC) these > methods are for an internal use only... what am I missing? TIA! :) > >> Not great: >> >> Coverage for DbUtils is 12%: >> http://people.apache.org/builds/commons/dbutils/1.5/RC1/site/cobertura/org.apache.commons.dbutils.DbUtils.html >> > > not so nice indeed, but if they are a non-blocking issue I would go > ahead with a RC2... unless someone else is stepping up for quickly > provide patches/checks in tests in a short while.
I'm working on a patch for this (always wanted to try out Mocktio), but I'll not be able to finish it till tomorrow evening. If the patch doesn't make it into dbutil 1.5 we'll have the coverage for 1.6 ;-) Benedikt > > Thanks a lot, have a nice day! > -Simo > > http://people.apache.org/~simonetripodi/ > http://simonetripodi.livejournal.com/ > http://twitter.com/simonetripodi > http://www.99soft.org/ > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
