Conversation
WalkthroughThe pull request updates tracing in the transaction storage insertion: attribute keys were renamed to a namespaced form and a new Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This attribute is standardize and setting it with a string messes with signoz. Signed-off-by: Sylvain Rabot <sylvain@formance.com>
115ba0f to
d6e5753
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release/v2.3 #1247 +/- ##
=============================================
Coverage 81.78% 81.78%
=============================================
Files 187 187
Lines 9059 9060 +1
=============================================
+ Hits 7409 7410 +1
+ Misses 1215 1212 -3
- Partials 435 438 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This attribute is standardize and setting it with a string messes with signoz.