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
In #9560 I started printing the suppressed exception, however it would be best to also report it to Sentry so we can see when we introduce regressions.
Due to the architecture of command and updater, this is a bit tricky.
One way that just occurred to me is to store those exceptions in an array of errors on the MessageBuilder, and have the updater check that array for entries and report them to Sentry after the MessageBuilder runs.
The text was updated successfully, but these errors were encountered:
@jakecoffman I think something like this could do the trick? It's a bit more generic than storing it on the MessageBuilder and this way we can reuse it across all of core where we might run into a similar situation where we don't want to raise on errors, but we do want to report them to Sentry
Code improvement description
In #9560 I started printing the suppressed exception, however it would be best to also report it to Sentry so we can see when we introduce regressions.
Due to the architecture of command and updater, this is a bit tricky.
One way that just occurred to me is to store those exceptions in an array of errors on the MessageBuilder, and have the updater check that array for entries and report them to Sentry after the MessageBuilder runs.
The text was updated successfully, but these errors were encountered: