Skip to content

Conversation

bestbeforetoday
Copy link
Member

Requires a minimum of Java 17 for compilation. The release target remains Java 11.

This change also enabled Java compiler linting and fails on linting errors. To address lint failures in the existing codebase, the bahviour of the org.hyperledger.fabric.Logger constructor has changed to avoid a potential 'this' escape. Any subclasses must now ensure that the parent logger is set explicitly after construction by calling setParent(Logger). No changes are required for code obtaining logger instances using the static Logger.getLogger(String) method.

Requires a minimum of Java 17 for compilation. The release target
remains Java 11.

This change also enabled Java compiler linting and fails on linting
errors. To address lint failures in the existing codebase, the bahviour
of the org.hyperledger.fabric.Logger constructor has changed to avoid a
potential 'this' escape. Any subclasses must now ensure that the parent
logger is set explicitly after construction by calling
setParent(Logger). No changes are required for code obtaining logger
instances using the static Logger.getLogger(String) method.

Signed-off-by: Mark S. Lewis <[email protected]>
Copy link

sonarqubecloud bot commented Oct 5, 2025

@bestbeforetoday bestbeforetoday marked this pull request as ready for review October 5, 2025 17:44
@bestbeforetoday bestbeforetoday requested a review from a team as a code owner October 5, 2025 17:44
@bestbeforetoday bestbeforetoday enabled auto-merge (squash) October 5, 2025 17:47
@bestbeforetoday bestbeforetoday merged commit a927030 into hyperledger:main Oct 6, 2025
10 checks passed
@bestbeforetoday bestbeforetoday deleted the junit6 branch October 7, 2025 08:48
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.

2 participants