Skip to content

Bump rain.metadata submodule for OrderBuilderStateV1 rename#436

Open
findolor wants to merge 1 commit intomainfrom
bump-rain-metadata-order-builder-state
Open

Bump rain.metadata submodule for OrderBuilderStateV1 rename#436
findolor wants to merge 1 commit intomainfrom
bump-rain-metadata-order-builder-state

Conversation

@findolor
Copy link
Contributor

@findolor findolor commented Feb 24, 2026

Dependent PRs

Motivation

The rain.metadata crate renamed DotrainGuiStateV1 to OrderBuilderStateV1 as part of a broader effort to remove GUI terminology from the codebase and use more descriptive naming. This submodule bump pulls in that rename so downstream consumers (rain.orderbook) can reference the updated types.

Solution

Bumped lib/rain.metadata submodule to the commit that includes the OrderBuilderStateV1 rename.

Checks

By submitting this for review, I'm confirming I've done the following:

  • made this PR as small as possible
  • unit-tested any new functionality
  • linked any relevant issues or PRs
  • included screenshots (if this involves a front-end change)

Summary by CodeRabbit

  • Chores
    • Updated an internal library pointer.
    • No user-visible or behavioral changes in this release.

@findolor findolor self-assigned this Feb 24, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 38e82e8 and 0c82fb4.

📒 Files selected for processing (1)
  • lib/rain.metadata
🚧 Files skipped from review as they are similar to previous changes (1)
  • lib/rain.metadata

📝 Walkthrough

Walkthrough

Updated the submodule pointer in lib/rain.metadata from commit 153ee9042b0d5c3d94ab1e5f9ead4b0486efc902 to a712031050a5ac65b79e3cfa817558812ed1945e; no code or configuration logic changed.

Changes

Cohort / File(s) Summary
Submodule Update
lib/rain.metadata
Changed submodule commit pointer from 153ee9042b0d5c3d94ab1e5f9ead4b0486efc902 to a712031050a5ac65b79e3cfa817558812ed1945e. No other edits.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A pointer hops from old to new,
A tiny change, a quiet cue,
No code disturbed, no logic bent,
Just a commit with soft intent,
I twitch my nose and celebrate you! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: bumping the rain.metadata submodule to include the OrderBuilderStateV1 rename, which is the primary objective of the PR.
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
  • Commit unit tests in branch bump-rain-metadata-order-builder-state

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.

1 participant