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

Update mbari_wec_all.yaml #51

Merged
merged 1 commit into from
May 2, 2023
Merged

Update mbari_wec_all.yaml #51

merged 1 commit into from
May 2, 2023

Conversation

andermi
Copy link
Collaborator

@andermi andermi commented May 2, 2023

something like this should fix the actuator_msgs issue

@github-actions
Copy link

github-actions bot commented May 2, 2023

PR Preview Action a7e697c
Preview removed because the pull request was closed.
2023-05-02 20:20 UTC

@hamilton8415
Copy link
Collaborator

Great, I will try it just before lunch I think. Thank you.

@quarkytale
Copy link
Contributor

quarkytale commented May 2, 2023

Yeah this should work as well, I'll give it a try.
Edit: Works for me

@mabelzhang
Copy link
Collaborator

Oh I see. Sorry I've not been following the threads closely enough.
Yeah. This works and is lighter weight. Let's go with this since that's the preference.

@mabelzhang
Copy link
Collaborator

I guess we'll have to re-trigger CI in the other repo after this is merged.

@andermi andermi merged commit a7e697c into main May 2, 2023
@andermi andermi deleted the andermi/freeze_ros_gz branch May 2, 2023 20:20
@quarkytale quarkytale linked an issue May 2, 2023 that may be closed by this pull request
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.

CI failing to detect 'ros-humble-actuator-msgs'
4 participants