-
Notifications
You must be signed in to change notification settings - Fork 118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup tracing with a typed configuration. #336
Conversation
Generated by 🚫 Danger Swift against 559655a |
fb5f682
to
508fe93
Compare
Codecov ReportBase: 34.30% // Head: 32.04% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #336 +/- ##
===========================================
- Coverage 34.30% 32.04% -2.26%
===========================================
Files 233 269 +36
Lines 15043 17764 +2721
Branches 8680 9759 +1079
===========================================
+ Hits 5160 5693 +533
- Misses 9736 11894 +2158
- Partials 147 177 +30
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
56ab9b6
to
50c5ee5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ah-mazing, nicely done 👍
cc54df2
to
559655a
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This change allows us to create tracing presents so that its easier to increase the logging as needed.
I've also added the
RUST_BACKTRACE
env var to the run scheme (but disabled by default) as that seems like it would be useful to easily toggle on and off.