Pankraz76

Issue #16781: `NoWhitespaceBefore` blind spot resulting in high failure rate

22

### Understanding the `NoWhitespaceBefore` Check I may be misunderstanding the `NoWhitespaceBefore` check, but my current understanding is that it should detect and flag any unwanted whitespace that appears before specific...

Issue #16919: Use Dedicated Keywords like `forbid` as alternative to `disallow`

12

https://checkstyle.org/checks/imports/importcontrol.html#ImportControl `Disallow`, unlike dedicated words such as `prevent`, `forbid`, or `avoid`, isn't a standalone term with a clear, self-contained meaning. **_It's formed by adding a prefix to another word, which...

giving error-prone like ruleset, as default built-in to run quick analyses with a default configuration (convention over configuration)

7

not preventing error prone simply leads to error: - https://github.com/apache/maven/pull/2373 - https://github.com/apache/maven/pull/2370 - https://github.com/apache/maven/pull/2372 - https://github.com/apache/maven/pull/2371 - https://github.com/apache/maven/pull/2371 - https://github.com/apache/maven/pull/2368 - https://pmd.github.io/pmd/pmd_rules_jsp_errorprone.html Rules to detect constructs that are either broken,...

implement IDE agnostic configuration with `editorconfig.org`

10

Addressing the comment and issue of `people use a lot of editors and very different versions`, which is a valid concern—one good approach to handling this is using [EditorConfig](https://editorconfig.org/). `EditorConfig...

resolve false negative suppression

3

add double-check for each suppression to avoid dead-lock suppression stuff thats not existing. like in: - https://github.com/apache/maven/pull/2369 Just like out whitewords plugin checks for key given but not used. The...

Statement lambda can be replaced with expression lambda: Create `AvoidBraces` by inverting `NeedBraces`

7

## Discussion on Lambda Expression Style Preference ### Current Situation Is there an opportunity to revisit this setting to evaluate its necessity? Alternatively, is there a specific justification for its...

Issue #16981: `Migrate to Java 21 - [noTrailingWhitespace] not aware of string block """`

3

Issue #16981: `Migrate to Java 21 - [noTrailingWhitespace] not aware of string block """` - #16981 - https://github.com/checkstyle/checkstyle/blob/35367fcf1374443520909aeacb3026ba6d08e994/src/test/java/com/puppycrawl/tools/checkstyle/TreeWalkerTest.java#L415 ```java @Test public void testRequiredTokenIsEmptyIntArray() throws Exception { final File file =...

[java] UnnecessaryImport false positive for multiline @see Javadoc

6

**Affects PMD Version:** 7.14.0 **Rule:** [UnnecessaryImport](https://docs.pmd-code.org/latest/pmd_rules_java_codestyle.html#unnecessaryimport) **Description:** false positive finding: ``` [WARNING] PMD Failure: io.quarkus.deployment.configuration.BuildTimeConfigBuilderCustomizer:3 Rule:UnnecessaryImport Priority:4 Unused import 'java.util.List'. [WARNING] PMD Failure: io.quarkus.deployment.configuration.BuildTimeConfigBuilderCustomizer:4 Rule:UnnecessaryImport Priority:4 Unused import 'java.util.Map'. [WARNING]...

PoC: Replace checkstyle with Spotless & palantir-java-format: A modern, lambda-friendly, 120 character Java formatter.

10

- https://github.com/pmd/pmd/issues/5777 just like quarkus having eclipse config in place make is the perfect candidate for auto fix static analysis with spotless. - https://github.com/projectnessie/nessie/pull/624 - https://github.com/quarkusio/quarkus/pull/48041 - https://github.com/spring-projects/spring-framework/pull/34978 - https://github.com/palantir/palantir-java-format...

- https://sonarsource.github.io/rspec/#/rspec/S1488 - https://github.com/apache/maven/pull/2404/files#r2113612293 - https://docs.openrewrite.org/recipes/staticanalysis/inlinevariable - https://github.com/checkstyle/checkstyle/pull/16488 **InlineVariable** **Proposed Category:** One of [Best Practices] **Description:** Inline variables when they are immediately used to return or throw. **Code Sample:** This...