ok done with my lunch
there are many simpel errors which i can fix fast. but i'm not looking
forward to fix them every appfuse project i will do.
so can i commit these kind of fixes??
tibi wrote:
> ok with the appfuse config file there are 'only' 178 errors ;D
>
> tibi wrote:
>
>> tibi wrote:
>>
>>
>>> i'm used to use checkstyle and pmd to push on writing nice code.
>>> ...
>>> i already added the request to the google summer school list.
>>>
>>>
>>>
>> i will put here parts of my pom file. just as a start:
>> first it is very irritating to not be able to build when there is some
>> stupid checkstyle error. which is the whole point of this. but i need a
>> backdore so i have this propertie:
>> <failsOnError>true</failsOnError>
>> with the -DfailsOnError=false option you can always build (handy for a
>> continues integration server with mvn site -DfailsOnError=false to see the
>> errors)
>>
>> checkstyle is this in report section:
>> <plugin>
>> <groupId>org.apache.maven.plugins</groupId>
>> <artifactId>maven-checkstyle-plugin</artifactId>
>> <configuration>
>> <configLocation>
>> <!-- config/sun_checks.xml
>> ${basedir}/doc/checkstyle-checker.xml
>> -->
>>
>> https://appfuse.dev.java.net/checkstyle.xml
>> </configLocation>
>> <failsOnError>${failsOnError}</failsOnError>
>> </configuration>
>>
>> </plugin>
>>
>> pdm is this in report section:
>> <plugin>
>> <artifactId>maven-pmd-plugin</artifactId>
>> <configuration>
>> <linkXref>true</linkXref>
>> <targetJdk>1.5</targetJdk>
>> </configuration>
>> </plugin>
>> and you need this in the build section:
>> <!-- check and fail on the pmd reports -->
>> <plugin>
>> <groupId>org.apache.maven.plugins</groupId>
>> <artifactId>maven-pmd-plugin</artifactId>
>> <version>2.3</version>
>> <executions>
>> <execution>
>> <goals>
>> <goal>check</goal>
>> <goal>cpd-check</goal>
>> </goals>
>> </execution>
>> </executions>
>> </plugin>
>> en cobertura:
>> <!-- Fail if there are not enough tests -->
>> <plugin>
>> <groupId>org.codehaus.mojo</groupId>
>> <artifactId>cobertura-maven-plugin</artifactId>
>> <configuration>
>> <check>
>> <branchRate>85</branchRate>
>> <lineRate>85</lineRate>
>> <haltOnFailure>${failsOnError}</haltOnFailure>
>> <!-- <totalBranchRate>85</totalBranchRate>-->
>> <!-- <totalLineRate>85</totalLineRate>-->
>> <!-- <packageLineRate>85</packageLineRate>-->
>> <!-- <packageBranchRate>85</packageBranchRate>-->
>> <regexes>
>> <regex>
>> <pattern>nl.tryllian.aperture.example.*</pattern>
>> <branchRate>50</branchRate>
>> <lineRate>80</lineRate>
>> </regex>
>> </regexes>
>> </check>
>> </configuration>
>> <executions>
>> <execution>
>> <phase>test</phase>
>> <goals>
>> <goal>check</goal>
>> </goals>
>> </execution>
>> </executions>
>> </plugin>
>>
>>
>>
>>
>
> ---------------------------------------------------------------------
> 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]
