Skip to content

fix: removed direct output to stderr and stdout in core library code#352

Merged
gkc merged 1 commit intotrunkfrom
add_slf4j_logger
Feb 4, 2026
Merged

fix: removed direct output to stderr and stdout in core library code#352
gkc merged 1 commit intotrunkfrom
add_slf4j_logger

Conversation

@akafredperry
Copy link
Collaborator

- What I did

  • Removes direct output to System.out and System.err in core library code.
  • Cleanup of examples module included

NOTE CLI classes have NOT been modified and neither has the examples module (to keep it simple)

closes #337

- How I did it

  • Added project Lombok (this is a compile time library that removes the need for Java boiler plate and includes slf4j logging)
  • Replaced all the instances in which the code was printing to System.out and System.err with log statements
  • Replaced printStack trace statements with log statements

- How to verify it

- Description for the changelog
replaced direct output to stderr and stdout with slf4j logging in core library code

… CLI and examples still use System.out/err, replaced with slf4j using lombok annotations
@akafredperry akafredperry requested a review from gkc February 4, 2026 19:37
@gkc gkc merged commit 0d6dbf3 into trunk Feb 4, 2026
4 checks passed
@gkc gkc deleted the add_slf4j_logger branch February 4, 2026 22:11
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.

Remove all stdout and stderr logging

2 participants