GitHub user kinow opened a pull request:

    https://github.com/apache/commons-pool/pull/3

    Fix checkstyle errors

    The RAT report was enabled and configured for build, but not for reporting. 
The licence header was being verified by both, but inconsistently. Checkstyle 
complained about spaces, which RAT did not report anything about.
    
    This pull request adds RAT report for reporting, with the same 
configuration (i.e. ignores) as for build. Also fixes legit checkstyle issues, 
where javadoc for some methods was missing.
    
    Also added a .gitignore because running `mvn clean site` would create both 
target and site-content, which git marked as new.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kinow/commons-pool-1 fix-checkstyle-errors

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/commons-pool/pull/3.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3
    
----
commit e0aa08051417ac8cf718deb419c6f938c35b7a45
Author: Bruno P. Kinoshita <brunodepau...@yahoo.com.br>
Date:   2017-10-29T10:28:55Z

    Fix checkstyle errors

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to