Skip to content

Conversation

arjo129
Copy link
Contributor

@arjo129 arjo129 commented Sep 12, 2024

Related to:

gazebosim/gazebo_test_cases#1640

This commit adds a really simple regression check to make sure we actually publish on the right topic.

gazebosim/gazebo_test_cases#1640

This commit adds a really simple regression check to make sure we
actually publish on the right topic.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
@arjo129 arjo129 requested a review from mjcarroll as a code owner September 12, 2024 14:53
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Sep 12, 2024
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
@arjo129 arjo129 requested a review from azeey September 12, 2024 15:06
@arjo129 arjo129 mentioned this pull request Sep 12, 2024
8 tasks
Copy link
Contributor

@azeey azeey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @arjo129 ! Just one minor comment.

arjo129 and others added 2 commits September 12, 2024 13:38
@arjo129 arjo129 enabled auto-merge (squash) September 12, 2024 22:11
@arjo129 arjo129 merged commit 0f16949 into gz-sim9 Sep 12, 2024
10 checks passed
@arjo129 arjo129 deleted the arjo/test/regression_for_log_recordgui branch September 12, 2024 22:42
@arjo129
Copy link
Contributor Author

arjo129 commented Aug 5, 2025

https://github.com/Mergifyio backport gz-sim8

@mergify
Copy link
Contributor

mergify bot commented Aug 5, 2025

backport gz-sim8

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Aug 5, 2025
* Adds a regression test for logplayback

gazebosim/gazebo_test_cases#1640

This commit adds a really simple regression check to make sure we
actually publish on the right topic.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

* typo fix

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

* Make atomic

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

---------

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
(cherry picked from commit 0f16949)
azeey added a commit that referenced this pull request Aug 13, 2025
* Adds a regression test for logplayback

gazebosim/gazebo_test_cases#1640

This commit adds a really simple regression check to make sure we
actually publish on the right topic.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

* typo fix

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

* Make atomic

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

---------

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
(cherry picked from commit 0f16949)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏛️ ionic Gazebo Ionic

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants