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

Replace setup bag with subset bag in rolling #4050

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Replace setup bag with subset bag in rolling #4050

merged 1 commit into from
Dec 5, 2023

Conversation

songyuc
Copy link
Contributor

@songyuc songyuc commented Dec 5, 2023

This pull request addresses a typographical error found in the ROS 2 Iron documentation, specifically in the "Reading from a bag file (C++)" tutorial. The tutorial erroneously references a "setup bag" when it should actually be "subset bag". This confusion was discussed and clarified in Issue #4048.

The correction ensures that the documentation accurately reflects the content of the tutorials, particularly where it guides users in generating and using bag files in ROS 2.

@songyuc
Copy link
Contributor Author

songyuc commented Dec 5, 2023

Hi, @fujitatomoya,

I want to let you know that I have made the suggested changes, and would greatly appreciate it if you could take another look at the new pull request and provide any further feedback or approval as needed.

Thank you for your time and guidance on this matter.

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

lgtm

@fujitatomoya fujitatomoya merged commit f4fd8dd into ros2:rolling Dec 5, 2023
3 checks passed
@fujitatomoya
Copy link
Collaborator

@Mergifyio backport to iron and humble

Copy link
Contributor

mergify bot commented Dec 5, 2023

backport to iron and humble

❌ No backport have been created

  • Backport to branch to failed

GitHub error: Branch not found

GitHub error: Branch not found

mergify bot pushed a commit that referenced this pull request Dec 5, 2023
mergify bot pushed a commit that referenced this pull request Dec 5, 2023
@songyuc songyuc deleted the new-feature-updated branch December 5, 2023 17:26
fujitatomoya pushed a commit that referenced this pull request Dec 5, 2023
(cherry picked from commit f4fd8dd)

Co-authored-by: songyuc <466309936@qq.com>
fujitatomoya pushed a commit that referenced this pull request Dec 5, 2023
(cherry picked from commit f4fd8dd)

Co-authored-by: songyuc <466309936@qq.com>
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.

2 participants