Skip to content

Pull #17891: Add id to NeedBraces declaration#17891

Merged
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:fix-NeedBraces
Oct 12, 2025
Merged

Pull #17891: Add id to NeedBraces declaration#17891
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:fix-NeedBraces

Conversation

@Pankraz76

@Pankraz76 Pankraz76 commented Oct 10, 2025

Copy link
Copy Markdown

Pull #17891: Remove duplicated NeedBraces declaration

@Pankraz76 Pankraz76 changed the title Pull #14194: Remove duplicated NeedBraces declaration. Pull #17891: Remove duplicated NeedBraces declaration. Oct 10, 2025
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Oct 10, 2025
Comment thread config/checkstyle-checks.xml
@Pankraz76 Pankraz76 marked this pull request as ready for review October 10, 2025 10:32
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Oct 10, 2025
@Pankraz76 Pankraz76 changed the title Pull #17891: Remove duplicated NeedBraces declaration. Pull #17891: Add id to NeedBraces declaration. Oct 10, 2025
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Oct 11, 2025
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Oct 11, 2025
@romani romani changed the title Pull #17891: Add id to NeedBraces declaration. Pull #17891: Add id to NeedBraces declaration Oct 11, 2025
@romani

romani commented Oct 11, 2025

Copy link
Copy Markdown
Member

please fix CI violation

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Oct 11, 2025

@romani romani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot

@romani romani merged commit bad6d8d into checkstyle:master Oct 12, 2025
120 checks passed
@github-actions github-actions Bot added this to the 12.0.2 milestone Oct 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants