Nick Mancuso
Nick Mancuso
@Vyom-Yadav please update PR description with all reports and link to check documentation
Closed via https://github.com/checkstyle/checkstyle/pull/12052
Supression of constructors with this annotation can be achieved via [SuppresionXpathSingleFilter](https://checkstyle.sourceforge.io/config_filters.html#SuppressionXpathSingleFilter): ``` ➜ java cat Test.java public class Test { private final String a1; private final String a2; private final...
As for me, I prefer to keep checks simple, and only add properties if they cannot be covered by some reasonable/ reliable XPath or other filter. So, > avoid extra...
After reading through the discussion here, and thinking on this some more, I have reconsidered my general position on check properties. If a new check property that is a “feature”...
Let's make a decision here, in regards to this particular issue. If we want to continue discussion about adding check properties in general, let's create a Github discussion and continue...
Please follow bug report template.
With slightly different formatting for your text block declaration, you can use `SuppressWithPlainTextCommentFilter` to suppress these warnings: ``` ➜ java git:(master) ✗ cat config.xml ➜ java git:(master) ✗ cat Test.java...
@rnveach after thinking on this a bit and reviewing your PR at https://github.com/checkstyle/checkstyle/pull/11652, I understand the motivation behind this issue. It does seem like it could be problematic to pull...
> @rnveach @romani The mutator has been added, can we close this? Please show proof via CLI with latest master branch (include `git log -1`) that all profiles include this...