Skip to content

Conversation

@Tsukilc
Copy link
Contributor

@Tsukilc Tsukilc commented Nov 12, 2025

Purpose of the PR

  • close #xxx

Main Changes

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • xxx

Does this PR potentially affect the following parts?

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. store Store module labels Nov 12, 2025
@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 31.90%. Comparing base (496b150) to head (71eb68a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2905      +/-   ##
============================================
- Coverage     35.55%   31.90%   -3.66%     
- Complexity      333      488     +155     
============================================
  Files           802      802              
  Lines         67564    67564              
  Branches       8776     8776              
============================================
- Hits          24024    21554    -2470     
- Misses        40984    43609    +2625     
+ Partials       2556     2401     -155     

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

Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: we need to ensure the root case. merge it first

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 12, 2025
@imbajin imbajin requested review from Pengzna and VGalaxies November 12, 2025 11:58
@github-project-automation github-project-automation bot moved this from In progress to In review in HugeGraph PD-Store Tasks Nov 12, 2025
@imbajin imbajin merged commit 41d0dbc into apache:master Nov 12, 2025
14 of 15 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in HugeGraph PD-Store Tasks Nov 12, 2025
@imbajin imbajin deleted the fix/reflect-auth branch November 12, 2025 11:58
hahahahbenny pushed a commit to hahahahbenny/incubator-hugegraph that referenced this pull request Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files. store Store module

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants