Refactor project structure and update README for clarity#2
Refactor project structure and update README for clarity#2cychitivav wants to merge 10 commits intomasterfrom
Conversation
cychitivav
commented
Mar 3, 2026
- Fix the README documentation
- Enhance tmux scripts
…age.xml and tmux scripts for consistency with new naming conventions.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ${mrs_msgs_TARGETS} | ||
| ${std_srvs_TARGETS} | ||
| ) | ||
|
|
There was a problem hiding this comment.
target_link_libraries() links against ${mrs_msgs_TARGETS}, but this CMakeLists.txt never calls find_package(mrs_msgs REQUIRED) (and the code in this package doesn’t appear to use mrs_msgs). As written, CMake configuration will fail due to an undefined variable/target list. Either add the missing find_package(mrs_msgs REQUIRED) + proper linking/export, or remove ${mrs_msgs_TARGETS} from the link list.
| ${mrs_msgs_TARGETS} | |
| ${std_srvs_TARGETS} | |
| ) | |
| ${std_srvs_TARGETS} | |
| ) |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Added sections for ROS interfaces including Messages, Services, and Actions in the README.
|
Please fix the following when running a tmux session:
Test it ;) Also functional changes shouldn't be in refactor README branch :D |