chore: upgrade Rust toolchain to 1.93.1 and release 0.5.1#14
chore: upgrade Rust toolchain to 1.93.1 and release 0.5.1#14soulgarden merged 2 commits intomainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughThis PR bumps the Rust toolchain to 1.93.1 across CI, toolchain, and Docker; upgrades multiple Cargo dependencies and the package VERSION to 0.5.1; condenses README content; and hardens an integration test shutdown sequence. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🤖 Fix all issues with AI agents
Verify each finding against the current code and only fix it if needed.
In `@Cargo.toml`:
- Around line 13-14: The Cargo.toml currently specifies both serde = { version =
"1.0.228", features = ["derive"] } and serde_derive = "1.0.228", which is
redundant because the "derive" feature on serde re-exports serde_derive; remove
the separate serde_derive dependency entry so only serde with features =
["derive"] remains to avoid duplication.
In `@README.md`:
- Line 90: Replace the hardcoded test count and date in the README line "Tests:
293 unit/integration tests (as of Feb 16, 2026)" with a maintenance-friendly
phrase; for example change to "Tests: ~300 unit/integration tests" or "Tests:
300+ unit/integration tests" or remove the date entirely so the README no longer
requires manual updates when tests change—locate the exact string in README.md
and update it accordingly.
🧹 Nitpick comments (2)
🤖 Fix all nitpicks with AI agents
Verify each finding against the current code and only fix it if needed. In `@Cargo.toml`: - Around line 13-14: The Cargo.toml currently specifies both serde = { version = "1.0.228", features = ["derive"] } and serde_derive = "1.0.228", which is redundant because the "derive" feature on serde re-exports serde_derive; remove the separate serde_derive dependency entry so only serde with features = ["derive"] remains to avoid duplication. In `@README.md`: - Line 90: Replace the hardcoded test count and date in the README line "Tests: 293 unit/integration tests (as of Feb 16, 2026)" with a maintenance-friendly phrase; for example change to "Tests: ~300 unit/integration tests" or "Tests: 300+ unit/integration tests" or remove the date entirely so the README no longer requires manual updates when tests change—locate the exact string in README.md and update it accordingly.Cargo.toml (1)
13-14: Nit:serde_deriveis redundant whenserdehasfeatures = ["derive"].The
derivefeature onserdealready re-exportsserde_derive. Having both is harmless but unnecessary. This is pre-existing, so no action needed in this PR — just flagging for awareness.♻️ Optional cleanup (can be done in a follow-up)
serde = { version = "1.0.228", features = ["derive"] } -serde_derive = "1.0.228"🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Cargo.toml` around lines 13 - 14, The Cargo.toml currently specifies both serde = { version = "1.0.228", features = ["derive"] } and serde_derive = "1.0.228", which is redundant because the "derive" feature on serde re-exports serde_derive; remove the separate serde_derive dependency entry so only serde with features = ["derive"] remains to avoid duplication.README.md (1)
90-90: Hardcoded test count and date will become stale.The specific count "293" and date "Feb 16, 2026" will require manual updates whenever tests are added/removed. Consider a more generic phrasing like "~300 unit/integration tests" or removing the date, to reduce maintenance burden.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 90, Replace the hardcoded test count and date in the README line "Tests: 293 unit/integration tests (as of Feb 16, 2026)" with a maintenance-friendly phrase; for example change to "Tests: ~300 unit/integration tests" or "Tests: 300+ unit/integration tests" or remove the date entirely so the README no longer requires manual updates when tests change—locate the exact string in README.md and update it accordingly.
Summary
1.93.1(rust-toolchain.toml, CI workflow, Docker builder image)0.5.1alpine:3.23Validation
cargo fmt --all -- --checkmake lintmake test(293 passed)make checkdocker build --platform linux/amd64 -f Dockerfile .Notes
soulgarden/logfowd2:0.5.1.Summary by CodeRabbit
Chores
Documentation
Tests