Skip to content
This repository was archived by the owner on Apr 2, 2025. It is now read-only.

fix exception logging #125

Merged
philvarner merged 3 commits into from
Jan 7, 2025
Merged

fix exception logging #125

philvarner merged 3 commits into from
Jan 7, 2025

Conversation

@philvarner
Copy link
Contributor

@philvarner philvarner commented Jan 7, 2025

Related Issue(s):

Proposed Changes:

  1. Fix exception logging

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@philvarner philvarner requested a review from jkeifer January 7, 2025 13:57
trevorskaggs
trevorskaggs approved these changes Jan 7, 2025
View reviewed changes
Copy link

@trevorskaggs trevorskaggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple, effective.

@philvarner philvarner merged commit 74a5d01 into main Jan 7, 2025
2 checks passed
@philvarner philvarner deleted the pv/fix-exception-logging branch January 7, 2025 16:19
@jkeifer jkeifer mentioned this pull request Jan 11, 2025
Merged
1 task
@pjhartzell pjhartzell mentioned this pull request Feb 8, 2025
Merged
1 task
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jkeifer jkeifer Awaiting requested review from jkeifer

1 more reviewer

@trevorskaggs trevorskaggs trevorskaggs approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@philvarner @trevorskaggs