Skip to content

Enforce spaces inside parentheses for control flow statements via checkstyle#4016

Merged
filiphr merged 2 commits intomapstruct:mainfrom
filiphr:checkstyle-parenpad-control-flow
Mar 27, 2026
Merged

Enforce spaces inside parentheses for control flow statements via checkstyle#4016
filiphr merged 2 commits intomapstruct:mainfrom
filiphr:checkstyle-parenpad-control-flow

Conversation

@filiphr
Copy link
Copy Markdown
Member

@filiphr filiphr commented Mar 15, 2026

No description provided.

@filiphr filiphr added this to the 1.7.0.Beta2 milestone Mar 15, 2026
@filiphr filiphr added the build label Mar 15, 2026
@hduelme
Copy link
Copy Markdown
Contributor

hduelme commented Mar 16, 2026

In #3994, Claude Code found a missing check for the import order. Maybe we could add this.

After merging, the new rules should also be applied to mapstruct-idea to prevent unexpected errors in future PRs.

@filiphr
Copy link
Copy Markdown
Member Author

filiphr commented Mar 16, 2026

Good point. That PR was a trigger for me to do this with Claude Code. I'll look into the import order as well

@filiphr filiphr force-pushed the checkstyle-parenpad-control-flow branch 2 times, most recently from 57bbb86 to f6d7d34 Compare March 27, 2026 17:02
@filiphr filiphr force-pushed the checkstyle-parenpad-control-flow branch from f6d7d34 to 3073dbb Compare March 27, 2026 20:34
Add CustomImportOrder module to checkstyle configuration enforcing:
1. java/javax imports (STANDARD_JAVA_PACKAGE)
2. Third-party imports (THIRD_PARTY_PACKAGE)
3. org.hibernate imports (SPECIAL_IMPORTS)
4. Static imports (STATIC)

With alphabetical sorting within groups and blank line separators
between groups, matching the existing IntelliJ formatter configuration.
@filiphr filiphr force-pushed the checkstyle-parenpad-control-flow branch from 3073dbb to 16a28e2 Compare March 27, 2026 20:59
@filiphr filiphr merged commit 99f9bcc into mapstruct:main Mar 27, 2026
5 checks passed
@filiphr filiphr deleted the checkstyle-parenpad-control-flow branch March 27, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants