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

Finish up API documentation #123

Merged
merged 2 commits into from
Jun 4, 2020
Merged

Finish up API documentation #123

merged 2 commits into from
Jun 4, 2020

Conversation

brawner
Copy link
Contributor

@brawner brawner commented Jun 3, 2020

This adds API documentation to each message and service that didn't already have them.

Signed-off-by: Stephen Brawner brawner@gmail.com

Signed-off-by: Stephen Brawner <brawner@gmail.com>
@brawner brawner requested review from ahcorde and Blast545 June 3, 2020 23:49
Copy link
Contributor

@Blast545 Blast545 left a comment

Choose a reason for hiding this comment

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

Just a minor suggestion

Comment on lines 5 to 7
# Single DOF joint positions for each joint relative to their "0" position.
# The units depend on the specific joint type: radians for revolute and
# continuous joints and meters for prismatic joints.
Copy link
Contributor

Choose a reason for hiding this comment

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

The repeated "and" makes it a bit confusing to read, maybe something like:

Suggested change
# Single DOF joint positions for each joint relative to their "0" position.
# The units depend on the specific joint type: radians for revolute and
# continuous joints and meters for prismatic joints.
# Single DOF joint positions for each joint relative to their "0" position.
# The units depend on the specific joint type: radians for revolute and
# continuous joints while meters for prismatic joints.

(or a comma separation as the one below this)

Signed-off-by: Stephen Brawner <brawner@gmail.com>
@brawner brawner merged commit ba32be2 into master Jun 4, 2020
@delete-merged-branch delete-merged-branch bot deleted the brawner/inline_documentation branch June 4, 2020 17:23
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

3 participants