refactor: add realsense to root flake - #3767
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #3767 +/- ##
==========================================
- Coverage 77.83% 77.83% -0.01%
==========================================
Files 1329 1329
Lines 126864 126865 +1
Branches 11034 11034
==========================================
- Hits 98750 98745 -5
- Misses 24962 24968 +6
Partials 3152 3152
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Greptile SummaryThe RealSense Rust module is now included in the root workspace and its Linux native dependencies are declared in the Nix environment. On Darwin, the development environment still omits Librealsense and its pkg-config metadata, so workspace-wide Cargo checks fail when building the newly included RealSense crate. Confidence Score: 4/5The change is not ready to merge for Darwin developers because root workspace Rust commands cannot resolve the newly required RealSense native dependency. One actionable non-security failure remains: the Darwin Nix shell excludes both the Librealsense library and its development metadata required by Files Needing Attention: flake.nix
What T-Rex did
|
Contribution path
Problem
realsense module was in a separate workspace so it could have it's own flake for librealsense. This also caused precommit to fail.
Solution
Consolidates the realsense module to the root crate and fixes the precommit
How to Test
AI assistance
Checklist