[ 
https://issues.apache.org/jira/browse/CRUNCH-15?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13414722#comment-13414722
 ] 

Matthias Friedrich edited comment on CRUNCH-15 at 7/15/12 5:53 PM:
-------------------------------------------------------------------

Rahul, I can't commit anything, so I attached my checkstyle config to this 
issue :)

Applying the patch works better now using "git am 0001-adding-reporting.patch", 
there are just a few whitespace warnings. You can catch this by running "git 
diff --check" after making changes to your working copy but before adding the 
modified files to the index. Also, you should configure your full name and mail 
address with git so that you are properly credited. I found these instructions 
helpful: https://help.github.com/articles/set-up-git
                
      was (Author: mafr):
    Rahul, I can't commit anything, so I attached my checkstyle config to this 
issue :)

Applying the patch works better now using "git am 0001-adding-reporting.patch", 
there are just a few whitespace warnings. You can catch this by running "git 
status --check" after making changes to your working copy but before adding the 
modified files to the index. Also, you should configure your full name and mail 
address with git so that you are properly credited. I found these instructions 
helpful: https://help.github.com/articles/set-up-git
                  
> Add reports to Maven site
> -------------------------
>
>                 Key: CRUNCH-15
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-15
>             Project: Crunch
>          Issue Type: Task
>            Reporter: Matthias Friedrich
>         Attachments: 0001-adding-reporting.patch, checkstyle.xml, 
> report-gen.patch
>
>
> Decide which reports are useful and add them to the site. Options:
>   * Cobertura
>   * Checkstyle
>   * PMD
>   * FindBugs
>   * JDepend
>   * Project info reports: 
> http://maven.apache.org/plugins/maven-project-info-reports-plugin/
> Note that this issue depends on CRUNCH-12 and that Checkstyle probably needs 
> extensive configuration.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to