New sniff to check for "long" closures and recommend using named functions instead.
The sniff is fully configurable via the following properties:
(int) recommendedLines - this determines when a warning will be thrown. Defaults to 5, which means that a warning will be thrown if the closure is more than 5 lines long.
(int) maxLines - this determines when an error will be thrown. Defaults to 8, which means that an error will be thrown if the closure is more than 8 lines long.
(bool) ignoreCommentLines - whether or not comment-only lines should be ignored for the lines count. Defaults to true.
(bool) ignoreEmptyLines - whether or not blank lines should be ignored for the lines count. Defaults to true.
The error and the warning have their own error codes - ExceedsMaximum and ExceedsRecommended - and can be included/excluded separately from a ruleset.
Includes unit tests.
Includes documentation.
Includes metrics.
New sniff to check for "long" closures and recommend using named functions instead.
The sniff is fully configurable via the following properties:
recommendedLines
- this determines when a warning will be thrown. Defaults to5
, which means that a warning will be thrown if the closure is more than 5 lines long.maxLines
- this determines when an error will be thrown. Defaults to8
, which means that an error will be thrown if the closure is more than 8 lines long.ignoreCommentLines
- whether or not comment-only lines should be ignored for the lines count. Defaults totrue
.ignoreEmptyLines
- whether or not blank lines should be ignored for the lines count. Defaults totrue
.The error and the warning have their own error codes -
ExceedsMaximum
andExceedsRecommended
- and can be included/excluded separately from a ruleset.Includes unit tests. Includes documentation. Includes metrics.
Closes #192