[ 
https://issues.apache.org/jira/browse/OPENJPA-2410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rick Curtis updated OPENJPA-2410:
---------------------------------

    Description: 
Source files may have inadvertent System.out/err.print statements left in 
source files on commit. Ideally we don't want any of these changes being 
checked in and a large majority of the time logging facilities should be 
utilized. That being said, there are a number of cases where we don't have 
access to a logger and must use System.out/err.

This JIRA will be used to update our checkstyle to scan for unwanted print 
statements, and fail the build when/if they are encountered.


  was:
OpenJPA may contain illegal System.out/err.print(ln) statements. Those 
statements should be detected and an error should be thrown to the user so they 
are aware of them and remove them from the code. Certain 
System.out/err.print(ln) statements are allowed such as for tools and testing 
purpose. Those will be filtered out and will not cause an error. 

Files modified:
checkstyle.xml
pom.xml 

File added:
suppressions.xml

    
> Build time detection of System.out/err.print(ln) in source files
> ----------------------------------------------------------------
>
>                 Key: OPENJPA-2410
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-2410
>             Project: OpenJPA
>          Issue Type: Improvement
>          Components: build / infrastructure
>    Affects Versions: 2.3.0
>            Reporter: Di Wu Lau
>            Assignee: Rick Curtis
>            Priority: Minor
>             Fix For: 2.3.0
>
>         Attachments: OPENJPA-2410.patch
>
>
> Source files may have inadvertent System.out/err.print statements left in 
> source files on commit. Ideally we don't want any of these changes being 
> checked in and a large majority of the time logging facilities should be 
> utilized. That being said, there are a number of cases where we don't have 
> access to a logger and must use System.out/err.
> This JIRA will be used to update our checkstyle to scan for unwanted print 
> statements, and fail the build when/if they are encountered.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to