Closed sweep-ai[bot] closed 1 year ago
✨ Created PR: https://github.com/pavlovcik/ubiquibot/pull/14 to fix All new business logic should have corresponding unit tests in the tests/ directory.
.
This PR was made against the sweep/refactor-logger
branch, not your main branch, so it's safe to merge if it looks good!
PR Feedback (click)
Description
This PR refactors the logger system in the Ubiquity GitHub Bot project to be event-based, making it easier to use and accessible on the
runtime.logger
object.Summary of Changes
getLogger
function insrc/bindings.ts
to return an event-based logger that emits events whenever a log message is generated.src/events/loggerEvents.ts
to define a set of events that the logger can emit.src/handlers/shared/handler.ts
and updated the usage of the logger in thenullHandler
function to subscribe to the DEBUG event and handle the event by logging the message.Please review and merge this PR to incorporate the event-based logger system into the project.
Fixes #12.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: