Skip to content

fix: gate Unix database hosts by platform - #11

Merged
dsecurity49 merged 1 commit into
mainfrom
windows-build
Aug 22, 2026
Merged

fix: gate Unix database hosts by platform#11
dsecurity49 merged 1 commit into
mainfrom
windows-build

Conversation

@dsecurity49

@dsecurity49 dsecurity49 commented Aug 22, 2026

Copy link
Copy Markdown
Owner

No description provided.

@dsecurity49
dsecurity49 merged commit 00c51ff into main Aug 22, 2026
7 of 8 checks passed
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f7232b0e-8002-478a-bd0f-93cf49fe6a35

📥 Commits

Reviewing files that changed from the base of the PR and between ff8a02d and 00c51ff.

📒 Files selected for processing (2)
  • src/sync.rs
  • tests/common/mod.rs

📝 Walkthrough

Walkthrough

The change adds Unix-only compilation guards to Host::Unix match arms in synchronization logic and shared test helpers. TCP host handling remains unchanged.

Changes

Unix host match gating

Layer / File(s) Summary
Guard Unix host matches
src/sync.rs, tests/common/mod.rs
The Host::Unix match arms compile only on Unix targets. TCP host handling remains unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch windows-build

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.

@dsecurity49
dsecurity49 deleted the windows-build branch August 22, 2026 13:06
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