Skip to content

chore(base-logging): move LogRecordPublisher from base-flow to base-logging - #96

Merged
deer merged 1 commit into
mainfrom
move_logrecordpublisher
Jul 17, 2026
Merged

chore(base-logging): move LogRecordPublisher from base-flow to base-logging#96
deer merged 1 commit into
mainfrom
move_logrecordpublisher

Conversation

@deer

@deer deer commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

LogRecordPublisher is a java.util.logging Handler that publishes LogRecords to base-flow subscribers, but it lived in base-flow, forcing that module to require java.logging for the sake of a single class that's really about logging integration, not flow itself.

This moves LogRecordPublisher into base-logging under the build.base.logging package, drops requires java.logging from base-flow/module-info.java, and adds build.base.flow as a compile-scope (previously test-scope) dependency of base-logging so it can still use Publisher, Subscriber, and SubscriberRegistry. base-logging/module-info.java now requires build.base.flow accordingly, and LoggerTests no longer needs the old build.base.flow.LogRecordPublisher import.

@deer
deer merged commit 1851493 into main Jul 17, 2026
3 checks passed
@deer
deer deleted the move_logrecordpublisher branch July 17, 2026 03:09
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.

1 participant