Skip to content

Conversation

kwonoj
Copy link
Owner

@kwonoj kwonoj commented Jul 23, 2025

Attempt to close #247. Largely borrowed from ai's approach so some things may slip through though.

@kwonoj kwonoj marked this pull request as ready for review July 23, 2025 05:57
@kwonoj kwonoj marked this pull request as draft July 23, 2025 06:03
@kwonoj kwonoj marked this pull request as ready for review July 23, 2025 06:03
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 0% with 29 lines in your changes missing coverage. Please review.

Project coverage is 45.01%. Comparing base (421435a) to head (7a541f4).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...e-instrument/src/macros/instrumentation_visitor.rs 0.00% 19 Missing ⚠️
...ument/src/macros/instrumentation_counter_helper.rs 0.00% 10 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@ ## main #273 +/- ## ========================================== - Coverage 45.35% 45.01% -0.34%  ========================================== Files 30 30 Lines 3896 3925 +29 ========================================== Hits 1767 1767 - Misses 2129 2158 +29 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@kwonoj kwonoj merged commit 75c5b18 into main Jul 23, 2025
1 of 3 checks passed
@kwonoj kwonoj deleted the fix-issues-247 branch July 23, 2025 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant