This is an automated email from the ASF dual-hosted git repository. aherbert pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-codec.git
commit ae667e0803fb96ae502bbc279ccd236dc1654631 Author: aherbert <[email protected]> AuthorDate: Fri Sep 4 12:40:38 2020 +0100 Add test sources to checkstyle:check --- checkstyle-suppressions.xml | 7 +++++++ pom.xml | 2 ++ 2 files changed, 9 insertions(+) diff --git a/checkstyle-suppressions.xml b/checkstyle-suppressions.xml index 4796ea0..f0400ca 100644 --- a/checkstyle-suppressions.xml +++ b/checkstyle-suppressions.xml @@ -24,4 +24,11 @@ limitations under the License. <suppressions> <suppress checks="Header" files="LICENSE.txt"/> <suppress checks="Header" files="NOTICE.txt"/> + <!-- Be more lenient on tests. --> + <suppress checks="Javadoc" files=".*[/\\]test[/\\].*" /> + <suppress checks="MultipleStringLiterals" files=".*[/\\]test[/\\].*" /> + <suppress checks="DesignForExtension" files=".*[/\\]test[/\\].*" /> + <suppress checks="LineLength" files=".*[/\\]test[/\\].*" /> + <suppress checks="IllegalCatch" files=".*[/\\]test[/\\].*" /> + <suppress checks="MethodLength" files="(MersenneTwister64|Well44497[ab]|MultiplyWithCarry256)Test" /> </suppressions> diff --git a/pom.xml b/pom.xml index fcf0685..91b1276 100644 --- a/pom.xml +++ b/pom.xml @@ -332,6 +332,7 @@ limitations under the License. <configLocation>${basedir}/checkstyle.xml</configLocation> <enableRulesSummary>false</enableRulesSummary> <headerFile>${basedir}/LICENSE-header.txt</headerFile> + <includeTestSourceDirectory>true</includeTestSourceDirectory> <resourceExcludes>NOTICE.txt,LICENSE.txt,**/pom.properties,**/sha512.properties</resourceExcludes> </configuration> </plugin> @@ -367,6 +368,7 @@ limitations under the License. <configLocation>${basedir}/checkstyle.xml</configLocation> <enableRulesSummary>false</enableRulesSummary> <headerFile>${basedir}/LICENSE-header.txt</headerFile> + <includeTestSourceDirectory>true</includeTestSourceDirectory> </configuration> <!-- We need to specify reportSets because 2.9.1 creates two reports --> <reportSets>
