Github user kinow commented on the issue:

    https://github.com/apache/commons-lang/pull/285
  
    Checked out pull request branch, built with the change, got no checkstyle 
errors. Then removed the suppression (without re-adding the previous value in 
pom.xml) and also got the same result, without any errors. Maybe it is aware 
that the suppression file must be ignored?
    
    +1 for the change. Will wait for others to review :+1: 
    
    Documentation about the SuppressionFilter module.
    
    http://checkstyle.sourceforge.net/config_filters.html#SuppressionFilter
    
    Thank you for your contribution @osiegmar 


---

Reply via email to