You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think it would be cleaner to add the level to the BudgetTrackingEvent itself, setting it by default to INFO and having error set it to ERROR. Then you won't need this class and can use the event publisher directly.
In the current implementation you don't have access to the full api of the emitter which is useful (see comment about the option to set the status by listening to events)
I think it would be cleaner to add the level to the BudgetTrackingEvent itself, setting it by default to INFO and having error set it to ERROR. Then you won't need this class and can use the event publisher directly.
In the current implementation you don't have access to the full api of the emitter which is useful (see comment about the option to set the status by listening to events)
Originally posted by @brafdlog in #110 (comment)
The text was updated successfully, but these errors were encountered: