Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable OpenRewrite's UnnecessaryParentheses #9876

Merged
merged 1 commit into from
May 11, 2023
Merged

Enable OpenRewrite's UnnecessaryParentheses #9876

merged 1 commit into from
May 11, 2023

Conversation

koppor
Copy link
Member

@koppor koppor commented May 10, 2023

Enables https://docs.openrewrite.org/recipes/java/cleanup/unnecessaryparentheses

Compulsory checks
Preview Give feedback

@koppor koppor added the type: code-quality Issues related to code or architecture decisions label May 10, 2023
@Siedlerchr
Copy link
Member

Siedlerchr commented May 11, 2023 via email

calixtus
calixtus approved these changes May 11, 2023
View reviewed changes
Copy link
Member

@calixtus calixtus left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scrolled through, looks good.
There are only very few places where braces were removed from conditions.

@koppor koppor merged commit c6ff4c1 into main May 11, 2023
@koppor koppor deleted the openrewrite-paranthethes branch May 11, 2023 07:40
@koppor koppor mentioned this pull request Jun 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calixtus calixtus calixtus approved these changes

@Siedlerchr Siedlerchr Siedlerchr approved these changes

Assignees
No one assigned
Labels
type: code-quality Issues related to code or architecture decisions
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@koppor @Siedlerchr @calixtus