Skip to content

chore: bump eloqstore submodule to bafb377#369

Merged
eatbreads merged 1 commit intoeloqdata:mainfrom
eatbreads:change_submodule
Jan 19, 2026
Merged

chore: bump eloqstore submodule to bafb377#369
eatbreads merged 1 commit intoeloqdata:mainfrom
eatbreads:change_submodule

Conversation

@eatbreads
Copy link
Collaborator

@eatbreads eatbreads commented Jan 19, 2026

Here are some reminders before you submit the pull request

  • Add tests for the change
  • Document changes
  • Reference the link of issue using fixes eloqdb/tx_service#issue_id
  • Reference the link of RFC if exists
  • Pass ./mtr --suite=mono_main,mono_multi,mono_basic

Summary by CodeRabbit

  • Chores
    • Updated internal infrastructure dependencies to the latest version.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Walkthrough

This pull request advances the submodule pointer for store_handler/eloq_data_store_service/eloqstore to a newer commit, reflecting upstream changes without modifying code or logic in the parent repository.

Changes

Cohort / File(s) Summary
Submodule Reference
store_handler/eloq_data_store_service/eloqstore
Updated commit pointer from bcb5c3f to bafb377

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • MrGuin
  • zhangh43
  • lzxddz

Poem

🐰✨ A pointer hops forward, light as can be,
The eloqstore submodule updates with glee!
From old commit hash to the new one so bright,
One tiny change, but it sets things right! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description only contains an unchecked checklist template with no additional context or explanation of the submodule bump. Add a brief explanation of why the submodule is being bumped, what improvements or fixes it includes, and reference any related issues using the template format.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: bumping the eloqstore submodule to a specific commit hash.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

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