Skip to content

Make the openable trait public#284

Open
irevoire wants to merge 1 commit intofjall-rs:mainfrom
irevoire:openable-trait-public
Open

Make the openable trait public#284
irevoire wants to merge 1 commit intofjall-rs:mainfrom
irevoire:openable-trait-public

Conversation

@irevoire
Copy link
Copy Markdown

@irevoire irevoire commented Apr 14, 2026

Hey!

That's a small PR to make the Openable trait public.
This would help me return a DatabaseBuilder<Self> in fiole here.

Summary by CodeRabbit

  • New Features
    • Expanded the public API to include additional transaction-related functionality, providing new capabilities for integration and use cases.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 729b67bf-80f4-41a6-9f05-a5d86d7d7972

📥 Commits

Reviewing files that changed from the base of the PR and between e36524a and 3b8f047.

📒 Files selected for processing (1)
  • src/lib.rs

📝 Walkthrough

Walkthrough

A public re-export of Openable from the tx::single_writer module was added to the library's public API, making this type accessible to external users alongside existing exports.

Changes

Cohort / File(s) Summary
Public API Expansion
src/lib.rs
Added public re-export of Openable from tx::single_writer to expose the type in the library's public API.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 clearly and concisely summarizes the main change: making the Openable trait public, which directly matches the single modification in the pull request.
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

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