Skip to content

Integrate with radare2 to build graph from binary - #19

Merged
hungpham10 merged 1 commit into
mainfrom
feature/integrate-with-radare2
Sep 6, 2026
Merged

Integrate with radare2 to build graph from binary#19
hungpham10 merged 1 commit into
mainfrom
feature/integrate-with-radare2

Conversation

@hungpham10

Copy link
Copy Markdown
Owner

No description provided.

@hungpham10 hungpham10 changed the title Integrate with radare2 to build codegraph from binary Integrate with radare2 to build graph from binary Sep 6, 2026
@coderabbitai

coderabbitai Bot commented Sep 6, 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: 804fc863-6809-4ed7-bc3a-730c7cead5fe


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.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.17949% with 70 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.27%. Comparing base (e9a7597) to head (090ce0d).

Files with missing lines Patch % Lines
crates/codegraph-binary/src/extract.rs 94.08% 20 Missing ⚠️
crates/codegraph-binary/src/lib.rs 43.75% 18 Missing ⚠️
crates/codegraph-binary/src/r2.rs 69.56% 14 Missing ⚠️
crates/codegraph-binary/src/cache.rs 86.44% 8 Missing ⚠️
crates/codegraph-binary/src/config.rs 61.90% 8 Missing ⚠️
crates/codegraph-binary/src/scan.rs 96.96% 1 Missing ⚠️
crates/codegraph/src/main.rs 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (87.17%) 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      #19      +/-   ##
==========================================
+ Coverage   73.83%   74.27%   +0.43%     
==========================================
  Files          73       79       +6     
  Lines       15231    15780     +549     
==========================================
+ Hits        11246    11720     +474     
- Misses       3985     4060      +75     

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

@hungpham10
hungpham10 merged commit b1d210d into main Sep 6, 2026
18 of 19 checks passed
@hungpham10
hungpham10 deleted the feature/integrate-with-radare2 branch September 6, 2026 15:59
@codspeed-hq

codspeed-hq Bot commented Sep 6, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks


Comparing feature/integrate-with-radare2 (090ce0d) with main (e9a7597)

Open in CodSpeed

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