-
Notifications
You must be signed in to change notification settings - Fork 0
Hey this looks pretty good #1
Copy link
Copy link
Closed as not planned
Description
.. but i have a suggestion, instead of using error stacktrace to find out the caller, it could be better to utilzie commandkit's context data (available in commands and events) to determine the caller at higher level. See how the internal logger does this https://github.com/underctrl-io/commandkit/blob/be5ac1f1fa49ee659a82ca438fdc265699755749/packages/commandkit/src/logger/DefaultLogger.ts#L78
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels