Skip to content
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

Merged
merged 11 commits into from
Jun 24, 2024
Merged

Conversation

sauk2
Copy link
Collaborator

@sauk2 sauk2 commented Jun 20, 2024

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.

@sauk2 sauk2 requested a review from j-rivero June 20, 2024 11:50
Copy link
Contributor

@j-rivero j-rivero left a 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 👏

.github/workflows/test.yml Show resolved Hide resolved
.github/workflows/test.yml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@sauk2
Copy link
Collaborator Author

sauk2 commented Jun 22, 2024

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 👏

@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 🚀

@j-rivero j-rivero merged commit 3d9a3d7 into gazebo-tooling:main Jun 24, 2024
13 checks passed
@sauk2 sauk2 deleted the feat/macos-install branch June 24, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement the action for MacOS platform
2 participants