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

3 to 4 #689

Merged
merged 3 commits into from
Mar 17, 2021
Merged

3 to 4 #689

merged 3 commits into from
Mar 17, 2021

Conversation

adlarkin
Copy link
Contributor

➡️ Forward port

Port ign-gazebo3 to ign-gazebo4

Branch comparison: ign-gazebo4...ign-gazebo3

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

chapulina and others added 2 commits March 15, 2021 13:36
Signed-off-by: Louise Poubel <louise@openrobotics.org>

Co-authored-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
…534)

Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Ashton Larkin <ashton@openrobotics.org>
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Mar 17, 2021
@adlarkin adlarkin merged commit f9a5ece into ign-gazebo4 Mar 17, 2021
@adlarkin adlarkin deleted the adlarkin/3_to_4 branch March 17, 2021 20:08
@azeey
Copy link
Contributor

azeey commented Mar 18, 2021

JointPositionControllerGui.Load is failing on CI https://build.osrfoundation.org/job/ignition_gazebo-ci-ign-gazebo4-bionic-amd64/54/

@adlarkin
Copy link
Contributor Author

JointPositionControllerGui.Load is failing on CI

It looks like this started to fail in the forward port, and I didn't catch it before merging 😬 sorry about that. @chapulina any thoughts on the failure?

@chapulina
Copy link
Contributor

Ahh let me take a look

@chapulina chapulina mentioned this pull request Mar 19, 2021
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants