pgjdbc/pgjdbc GitHub issues and pull requests (mirror)
help / color / mirror / Atom feedFrom: Praveen7294 (@Praveen7294) <[email protected]>
To: pgjdbc/pgjdbc <[email protected]>
Subject: [pgjdbc/pgjdbc] issue #3923: Formatting issue detected by updated Checkstyle EmptyLineSeparator check
Date: Mon, 02 Feb 2026 09:59:25 +0000
Message-ID: <[email protected]> (raw)
The pgjdbc project is currently used as a test input dependency in Checkstyle CI (CircleCI). While working on Checkstyle, I recently enhanced the `EmptyLineSeparatorCheck` to include handling for `ENUM_CONSTANT_DEF` token.
After this improvement, an existing formatting issue in the pgjdbc codebase is now detected by Checkstyle.
### Affected location:
`PGProperty.java`, around line 756
https://github.com/pgjdbc/pgjdbc/blob/1ac8d99a0042dd8250475aed477bdbfed565a7c1/pgjdbc/src/main/java/...
### Details:
According to the `EmptyLineSeparator` rule, there should not be more than one empty line before a block comment. The enum constant definition in this file violates this rule.
Previously, this violation was not reported because `ENUM_CONSTANT_DEF` token was not covered by the Checkstyle check. With the updated implementation, the violation is now correctly flagged.
### Why this matters:
Since pgjdbc is used in Checkstyle CI as an external test project, this violation may cause Checkstyle builds to fail when newer versions of Checkstyle are used. Aligning the code with the rule helps maintain compatibility and avoids CI noise.
### Suggested change:
Remove the extra empty line before the block comment in the enum constant definition to comply with the `EmptyLineSeparator` rule.
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: github://pgjdbc/pgjdbc
Cc: [email protected], [email protected]
Subject: Re: [pgjdbc/pgjdbc] issue #3923: Formatting issue detected by updated Checkstyle EmptyLineSeparator check
In-Reply-To: <<[email protected]>>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox