Skip to content

Conversation

andrewhop
Copy link
Contributor

Issues:

Addresses #2459

Description of changes:

Add more missing CFI directives.

Call-outs:

Point out areas that need special attention or support during the review process. Discuss architecture or design changes.

Testing:

Before:
image

After:
image

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@andrewhop andrewhop requested a review from a team as a code owner August 22, 2025 08:03
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.71%. Comparing base (67ef2bd) to head (f69cd47).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2634   +/-   ##
=======================================
  Coverage   78.71%   78.71%           
=======================================
  Files         646      646           
  Lines      111213   111213           
  Branches    15711    15711           
=======================================
  Hits        87544    87544           
- Misses      22975    22976    +1     
+ Partials      694      693    -1     

☔ 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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants