-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implemented action for MacOS #22
Conversation
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Saurabh Kamat <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can only say that the work is awesome! It works perfectly, can not find anything solid to be changed. 🚀
I left some minor suggestions and comments but the code is ready to go in. Congrats @sauk2 👏
Signed-off-by: Saurabh Kamat <[email protected]>
@j-rivero Thank you for your always helpful feedback! I have applied the changes you suggested. All checks pass! This PR is ready to be merged 🚀 |
Closes #18
Summary
This action can now be used to install Gazebo on MacOS. Additionally, updated the usage section of the README and added associated unit tests.
Added a job in
test.yml
to test the working of this action. This job installs Gazebo Harmonic on MacOS 12 (Monterey) and 13 (Ventura), as mentioned here.