Bump bitfields from 1.0.3 to 3.0.0#134
Conversation
Migrate packed entry structs to bitfields v3 API. The macro now auto-derives Clone and Copy, so manual derives for those conflict. PartialEq, Eq, and Hash are still added manually to preserve the public API surface of the affected types.
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
WalkthroughThis PR upgrades the 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Migrates the four packed entry structs in
reloaded-code-coreto the bitfields v3 API and closes the equivalent dependabot PR (#133).bitfieldsv3 auto-derivesCloneandCopyon the generated type, so the manual#[derive(Clone, Copy, ...)]lines on the four structs were dropped.PartialEq,Eq, andHashare still added manually to preserve the public API surface (PackedModelEntryis used as anAHashMapkey inbuilder.rs).Verified locally with
bash src/.cargo/verify.sh(build, test, clippy, docs all pass).