-
Couldn't load subscription status.
- Fork 409
Fix overriding of install #510
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
c375281 to
e416c5f
Compare
Signed-off-by: Tyler Weaver <[email protected]>
e416c5f to
c4b50d1
Compare
Co-authored-by: Chris Thrasher <[email protected]>
Co-authored-by: Chris Thrasher <[email protected]>
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #510 +/- ##
==========================================
- Coverage 35.78% 32.48% -3.31%
==========================================
Files 189 7 -182
Lines 17570 665 -16905
Branches 11592 357 -11235
==========================================
- Hits 6287 216 -6071
+ Misses 994 157 -837
+ Partials 10289 292 -9997
Flags with carried forward coverage won't be shown. Click here to find out more.
|
* Fix overriding of install Signed-off-by: Tyler Weaver <[email protected]> * Update admittance_controller/CMakeLists.txt Co-authored-by: Chris Thrasher <[email protected]> * Update ros2_controllers/CMakeLists.txt Co-authored-by: Chris Thrasher <[email protected]> * Remove unused lists * Use THIS_PACKAGE_INCLUDE_DEPENDS list --------- Signed-off-by: Tyler Weaver <[email protected]> Co-authored-by: Chris Thrasher <[email protected]>
This is related to:
Here are the colcon docs on overriding: https://colcon.readthedocs.io/en/released/user/overriding-packages.html#how-to-make-it-easier-for-your-users-to-override
Here is the ament_cmake user guide: https://docs.ros.org/en/rolling/How-To-Guides/Ament-CMake-Documentation.html