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

adding basic usage to description #18

Merged
merged 1 commit into from Aug 7, 2015
Merged

adding basic usage to description #18

merged 1 commit into from Aug 7, 2015

Conversation

tfoote
Copy link
Member

@tfoote tfoote commented Jul 31, 2015

Connects to ros2/ros2#76

@tfoote tfoote added in progress Actively being worked on (Kanban column) in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jul 31, 2015
@dirk-thomas
Copy link
Contributor

I think putting documentation like this into the package manifest will not be discoverable by users. That would be the last location I would look for information like this.

I would propose to either create a doc folder within the package and place a .md / .rst file there (as it is done for other packages: https://github.com/ament/ament_cmake/tree/master/ament_cmake_core) or create a README file in the package folder.

@tfoote
Copy link
Member Author

tfoote commented Aug 3, 2015

As someone entering a ROS codebase for the first time, my entry point is always the package.xml/manifest.xml.

Although it is basically API/usage documentation we do not have any standard location for that yet. The official package description is much more visible than a random markdown file embedded inside a package. Longer term I agree we should move it out into documentation. But since this package doesn't have even a full paragraph description I think it's more useful to new users browsing the packages to find this when they look for the description. It make it very clear what the package does and how to use it instead of a very abstract single sentence summary.

@esteve
Copy link
Contributor

esteve commented Aug 4, 2015

+1

@dirk-thomas
Copy link
Contributor

After discussing it we will put it in a markdown file in the doc folder for now.

@tfoote tfoote force-pushed the gtest_docs branch 2 times, most recently from f2a7abb to 4bd7087 Compare August 4, 2015 22:48
@tfoote
Copy link
Member Author

tfoote commented Aug 4, 2015

Updated to be in markdown in doc folder


To use this package do the following things.
- add a test dependency to your packages.xml: `<test_depend>ament_cmake_gtest</test_depend>`
- Write a gtest executable: Use `#include "gtest/gtest.h"` and standard gtest methods.
Copy link
Contributor

Choose a reason for hiding this comment

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

Use #include <...> instead.

@tfoote
Copy link
Member Author

tfoote commented Aug 4, 2015

typos fixed

@dirk-thomas
Copy link
Contributor

+1

tfoote added a commit that referenced this pull request Aug 7, 2015
adding basic usage to description
@tfoote tfoote merged commit 4f58214 into master Aug 7, 2015
@tfoote tfoote removed the in review Waiting for review (Kanban column) label Aug 7, 2015
@tfoote tfoote deleted the gtest_docs branch August 7, 2015 01:03
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