Skip to content

Improve code to be more maintainable - #18

Merged
hungpham10 merged 3 commits into
mainfrom
feature/refactor-codegraph-storage-to-make-everything-maintainable
Sep 4, 2026
Merged

Improve code to be more maintainable#18
hungpham10 merged 3 commits into
mainfrom
feature/refactor-codegraph-storage-to-make-everything-maintainable

Conversation

@hungpham10

Copy link
Copy Markdown
Owner

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 4, 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: 41f0f144-791f-4c21-80c6-517a51e7c411


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

codecov-commenter commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.30137% with 120 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.81%. Comparing base (a2b3434) to head (9e48095).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/codegraph-graph/src/storage/in_memory.rs 82.90% 67 Missing ⚠️
crates/codegraph-graph/src/storage.rs 47.05% 18 Missing ⚠️
crates/codegraph-graph/src/lib.rs 94.01% 10 Missing ⚠️
crates/codegraph-graph/src/shared.rs 30.00% 7 Missing ⚠️
crates/codegraph-graph/src/storage/redis.rs 72.72% 6 Missing ⚠️
crates/codegraph-graph/src/storage/mysql.rs 71.42% 4 Missing ⚠️
crates/codegraph-graph/src/storage/postgres.rs 71.42% 4 Missing ⚠️
crates/codegraph-graph/src/radix.rs 96.77% 2 Missing ⚠️
crates/codegraph-context/src/lib.rs 98.76% 1 Missing ⚠️
crates/codegraph-graph/src/storage/sqlite.rs 97.05% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (86.30%) 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      #18      +/-   ##
==========================================
+ Coverage   73.27%   73.81%   +0.54%     
==========================================
  Files          72       73       +1     
  Lines       15142    15231      +89     
==========================================
+ Hits        11095    11243     +148     
+ Misses       4047     3988      -59     

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

@codspeed-hq

codspeed-hq Bot commented Sep 4, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks


Comparing feature/refactor-codegraph-storage-to-make-everything-maintainable (901a40d) with main (a2b3434)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (c1354fb) during the generation of this report, so a2b3434 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@hungpham10
hungpham10 force-pushed the feature/refactor-codegraph-storage-to-make-everything-maintainable branch from d236f73 to 901a40d Compare September 4, 2026 22:54
@hungpham10
hungpham10 merged commit a7c46c5 into main Sep 4, 2026
19 of 20 checks passed
@hungpham10
hungpham10 deleted the feature/refactor-codegraph-storage-to-make-everything-maintainable branch September 4, 2026 23:29
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