Skip to content

Issue #18614: Add regression test for indentation of constructor parameters inside try block#18960

Merged
romani merged 1 commit into
checkstyle:masterfrom
MiniPiku:issue-18614-indentation-try-ctor-test
Feb 21, 2026
Merged

Issue #18614: Add regression test for indentation of constructor parameters inside try block#18960
romani merged 1 commit into
checkstyle:masterfrom
MiniPiku:issue-18614-indentation-try-ctor-test

Conversation

@MiniPiku

Copy link
Copy Markdown
Contributor

Closes #18614

The issue reported a false positive indentation violation
for constructor parameters inside a try block.

The issue is no longer reproducible in the current version of checkstyle-13.3.0-all.jar.
This PR adds a regression test to ensure the behavior
does not reappear in the future.

The test verifies that no indentation violations are
reported for properly indented constructor parameters
inside a try block.

@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.
Ok to merge if CI pass

@MiniPiku MiniPiku force-pushed the issue-18614-indentation-try-ctor-test branch from c9ea7b2 to 74fe724 Compare February 19, 2026 05:21
@MiniPiku

Copy link
Copy Markdown
Contributor Author

@romani
All CI checks are passing now.
Ready for merge when convenient.

@romani romani merged commit 28e7160 into checkstyle:master Feb 21, 2026
123 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False positive: indentation inside of constructor parameters inside try block

2 participants