Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update readme #60

Merged
merged 2 commits into from
Aug 18, 2021
Merged

Conversation

NirobNabil
Copy link
Contributor

@NirobNabil NirobNabil commented Aug 18, 2021

Tried to fix some issues i faced while installing the package. The readme didnt seem to provide sufficient information for inexperienced users like me on installing the package. Created a patch for the readme file including the steps i had to perform to get the package running on my system.
Fixes issues #30 and #55

- Added chmod command to make `install_usv_sim` script executable - Issue#55
- Added a rosdep command to install the required make dependencies  - Issue#30
@amamory amamory merged commit fbd3218 into disaster-robotics-proalertas:master Aug 18, 2021
@amamory
Copy link
Member

amamory commented Aug 18, 2021

thanks for the update !

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants