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

docs improvements #200

Merged
merged 5 commits into from
Sep 5, 2024
Merged

docs improvements #200

merged 5 commits into from
Sep 5, 2024

Conversation

adamdbrw
Copy link
Member

@adamdbrw adamdbrw commented Sep 5, 2024

Purpose

Improve documentation overall.

Proposed Changes

Fixes typos, rewords towards active, to-the-point and actionable phrasing, removes some obsolete parts, elaborates and re-structures other parts.

Copy link
Member

@boczekbartek boczekbartek left a comment

Choose a reason for hiding this comment

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

@adamdbrw I suggested adding a couple more links, but overall very nice improvement to docs. Thank you!

  • please run: pre-commit run -a to apply formatting ;)

docs/human_robot_interface.md Outdated Show resolved Hide resolved
docs/human_robot_interface.md Outdated Show resolved Hide resolved
docs/ros_packages.md Outdated Show resolved Hide resolved
adamdbrw and others added 4 commits September 5, 2024 14:48
Signed-off-by: Adam Dąbrowski <[email protected]>
Signed-off-by: Adam Dąbrowski <[email protected]>
@adamdbrw adamdbrw force-pushed the documentation_improvements branch from 7ce625c to 02ea1d5 Compare September 5, 2024 12:50
Signed-off-by: Adam Dąbrowski <[email protected]>
@adamdbrw adamdbrw merged commit cec8948 into development Sep 5, 2024
2 of 4 checks passed
@adamdbrw adamdbrw deleted the documentation_improvements branch September 5, 2024 12:53
maciejmajek pushed a commit that referenced this pull request Sep 18, 2024
Signed-off-by: Adam Dąbrowski <[email protected]>
Co-authored-by: Bartek Boczek <[email protected]>
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