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

Minor fixes in documentation (example_1 and example_12) #361

Merged
merged 2 commits into from
Sep 26, 2023

Conversation

olivier-stasse
Copy link
Collaborator

This PR is fixing simple stuff on top of the amazing recent work on documenting this repo.

In example 1, the first step is optional and RViz should be closed, and the second step start also RViz. Plus there is two in as some point.

In example 12, the file README.md is giving the wrong title for the example.

Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! The hint to the optional first step could be included to the other examples as well.

@christophfroehlich christophfroehlich merged commit 40b92dd into ros-controls:master Sep 26, 2023
18 checks passed
@christophfroehlich
Copy link
Contributor

@Mergifyio backport humble

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2023

backport humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Sep 26, 2023
* [example_1/doc] Specify that step 1 is optional.
Two in

* [example_12/README.md] Fix wrong title in README.md

---------

Co-authored-by: Olivier Stasse <olivier.stasse@laas.fr>
(cherry picked from commit 40b92dd)
christophfroehlich pushed a commit that referenced this pull request Sep 26, 2023
* [example_1/doc] Specify that step 1 is optional.
Two in

* [example_12/README.md] Fix wrong title in README.md

---------

Co-authored-by: Olivier Stasse <olivier.stasse@laas.fr>
(cherry picked from commit 40b92dd)

Co-authored-by: Olivier Stasse <ostasse@laas.fr>
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.

None yet

2 participants