Skip to content

Fix issue crash when analyzing binary and missing symbol function - #21

Open
hungpham10 wants to merge 3 commits into
mainfrom
fix/issue-crash-when-analysing-binary-but-not-recoginize-function
Open

Fix issue crash when analyzing binary and missing symbol function#21
hungpham10 wants to merge 3 commits into
mainfrom
fix/issue-crash-when-analysing-binary-but-not-recoginize-function

Conversation

@hungpham10

Copy link
Copy Markdown
Owner

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 4893ad63-2999-4785-a6f2-deda772b2e47


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codspeed-hq

codspeed-hq Bot commented Sep 7, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks


Comparing fix/issue-crash-when-analysing-binary-but-not-recoginize-function (88409ca) with main (503075a)

Open in CodSpeed

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.18519% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.40%. Comparing base (503075a) to head (88409ca).

Files with missing lines Patch % Lines
crates/codegraph-binary/src/extract.rs 85.00% 6 Missing ⚠️
crates/codegraph/src/main.rs 0.00% 2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (85.18%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   74.40%   74.40%   -0.01%     
==========================================
  Files          79       79              
  Lines       15872    15880       +8     
==========================================
+ Hits        11810    11815       +5     
- Misses       4062     4065       +3     

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants