Merge pull request #103 from kemitix/lambda-parameter-name

[ruleset] Add rule `LambdaParameterName`

CHANGELOG

@@ -1,9 +1,10 @@
 CHANGELOG
 =========
 
-4.1.2
+4.2.0
 -----
 
+* [ruleset] Add rule `LambdaParameterName`
 * Upgrade checkstyle to 8.12
 * Upgrade `sevntu` to 1.32.0
 * Switch to trunk-based development

README.md

@@ -167,6 +167,7 @@ Rule|Level|Source|Enabled|Suppressible
 [JavadocType](#javadoctype)|javadoc|checkstyle|Yes|
 [JavadocVariable](#javadocvariable)|javadoc|checkstyle||
 [JavaNCSS](#javancss)|complexity|checkstyle|Yes|
+[LambdaParameterName](#lambdaparametername)|naming|checkstyle|Yes|
 [LeftCurly](#leftcurly)|layout|checkstyle|Yes|
 [LineLength](#linelength)|layout|checkstyle|Yes|
 [LocalFinalVariableName](#localfinalvariablename)|naming|checkstyle|Yes|
@@ -1159,6 +1160,11 @@ Checks the format for Javadoc for classes and enums. Javadoc must be present, no
 Restricts the NCSS score for methods, classes and files to 40, 1200 and 1600 respectively. The NCSS score is a measure of the number of statements within a scope.
 
 Too high an NCSS score suggests that the method or class is doing too much and should be decomposed into smaller units.
+#### [LambdaParameterName](http://checkstyle.sourceforge.net/config_naming.html#LambdaParameterName)
+
+Checks the format of lambda parameter names.
+
+Identifiers must match `^[a-z][a-zA-Z0-9]*$`.
 #### [LeftCurly](http://checkstyle.sourceforge.net/config_blocks.html#LeftCurly)
 
 Checks that the left curly brace ('{') is placed at the end of the line. Does not check enums.

builder/src/main/resources/application.yml

@@ -1067,6 +1067,13 @@ rules:
         enabled: true
         source: SEVNTU
         uri: http://sevntu-checkstyle.github.io/sevntu.checkstyle/apidocs/com/github/sevntu/checkstyle/checks/design/ForbidWildcardAsReturnTypeCheck.html
+    -
+        name: LambdaParameterName
+        parent: TREEWALKER
+        level: NAMING
+        enabled: true
+        source: CHECKSTYLE
+        uri: http://checkstyle.sourceforge.net/config_naming.html#LambdaParameterName
     -
         name: LogicConditionNeedOptimization
         parent: TREEWALKER

builder/src/main/resources/rules/LambdaParameterName.md

@@ -0,0 +1,4 @@
+
+Checks the format of lambda parameter names.
+
+Identifiers must match `^[a-z][a-zA-Z0-9]*$`.

ruleset/src/main/resources/net/kemitix/checkstyle-2-naming.xml

@@ -85,6 +85,7 @@
 <module name="com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.naming.EnumValueNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.ForbidCCommentsInMethodsCheck"/>
+<module name="com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NameConventionForJunit4TestClassesCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NumericLiteralNeedsUnderscoreCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.SimpleAccessorNameNotationCheck"/>

ruleset/src/main/resources/net/kemitix/checkstyle-3-javadoc.xml

@@ -109,6 +109,7 @@
 <module name="com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.naming.EnumValueNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.ForbidCCommentsInMethodsCheck"/>
+<module name="com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NameConventionForJunit4TestClassesCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NumericLiteralNeedsUnderscoreCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.SimpleAccessorNameNotationCheck"/>

ruleset/src/main/resources/net/kemitix/checkstyle-4-tweaks.xml

@@ -158,6 +158,7 @@
 <module name="com.github.sevntu.checkstyle.checks.coding.EitherLogOrThrowCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.naming.EnumValueNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.ForbidCCommentsInMethodsCheck"/>
+<module name="com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.LogicConditionNeedOptimizationCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NameConventionForJunit4TestClassesCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.design.NoMainMethodInAbstractClassCheck"/>

ruleset/src/main/resources/net/kemitix/checkstyle-5-complexity.xml

@@ -208,6 +208,7 @@
 <module name="com.github.sevntu.checkstyle.checks.coding.ForbidCCommentsInMethodsCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.ForbidReturnInFinallyBlockCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.design.ForbidWildcardAsReturnTypeCheck"/>
+<module name="com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.LogicConditionNeedOptimizationCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.MapIterationInForEachLoopCheck"/>
 <module name="com.github.sevntu.checkstyle.checks.coding.NameConventionForJunit4TestClassesCheck"/>