Skip to content

Allow flexible constructor bodies#1069

Open
findepi wants to merge 1 commit intoairlift:masterfrom
findepi:findepi/allow-flexible-constructor-bodies-5a5ad2
Open

Allow flexible constructor bodies#1069
findepi wants to merge 1 commit intoairlift:masterfrom
findepi:findepi/allow-flexible-constructor-bodies-5a5ad2

Conversation

@findepi
Copy link
Contributor

@findepi findepi commented Jan 29, 2026

Java 25 adds ability to place statements within constructor before call to super(). Checkstyle does not support this yet, so it's configured not to fail the build on these.

Java 25 adds ability to place statements within constructor before call
to `super()`. Checkstyle does not support this yet, so it's configured
not to fail the build on these.
@electrum
Copy link
Member

electrum commented Feb 2, 2026

This means we don't get any style checks for these files. Is it so important to use this feature before checkstyle supports it?

@findepi
Copy link
Contributor Author

findepi commented Feb 3, 2026

This means we don't get any style checks for these files.

Correct

Is it so important to use this feature before checkstyle supports it?

If checkstyle adds support for this soon, it doesn't matter.
If checkstyle doesn't add support for this soon, do we want to be held back?

Checkstyle is advisory only anyway. The fact the code passes checkstyle doesn't unfortunately prove it won't change on Intellij's reformat, which is quite useful property to have.

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.

3 participants