chore: move fuzz profile settings to workspace root#501
chore: move fuzz profile settings to workspace root#501xdustinface wants to merge 1 commit intov0.42-devfrom
fuzz profile settings to workspace root#501Conversation
Fix the long existing cargo warning: ``` warning: profiles for the non root package will be ignored, specify profiles at the workspace root: package: /Users/kevin/GIT/rust-dashcore/fuzz/Cargo.toml workspace: /Users/kevin/GIT/rust-dashcore/Cargo.toml ``` Profile settings in non-root packages are ignored by Cargo. Move `codegen-units` to the workspace root's per-package profile override and remove the redundant `[profile.release]` section from `fuzz/Cargo.toml`. The `lto` and `panic` settings already match the workspace defaults.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe pull request modifies the build configuration for the dash-fuzz package. The main Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v0.42-dev #501 +/- ##
==========================================
Coverage 66.83% 66.84%
==========================================
Files 313 313
Lines 64878 64878
==========================================
+ Hits 43363 43366 +3
+ Misses 21515 21512 -3
*This pull request uses carry forward flags. Click here to find out more. 🚀 New features to boost your workflow:
|
Fix the long existing cargo warning:
Profile settings in non-root packages are ignored by Cargo. Move
codegen-unitsto the workspace root's per-package profile override and remove the redundant[profile.release]section fromfuzz/Cargo.toml. Theltoandpanicsettings already match the workspace defaults.Summary by CodeRabbit