-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Link to the API in the examples #4985
Comments
That indeed sounds like a good improvement! A pull request would be appreciated 😃
This is how we deal with that: https://docs.ros.org/en/rolling/The-ROS2-Project/Contributing/Contributing-To-ROS-2-Documentation.html#macros |
I don't disagree with this issue in principle, but back-filling most referenced objects with the API documentation is a pretty heavy lift. The API docs could stand significant improvement before we spend time back-filling the docs. |
If you sort by most common and only do examples, I think that the amount to fix is manageable. I would also add a rule that new docs have to be linked to APIs. |
hello team, |
@vimal0athithan of course you can, thanks! |
We should probably get #5050 merged before this, since it adds macros to link to interface (e.g., message) docs. |
I think APIs should be linked to in the examples in the docs.
For example in Writing a simple publisher and subscriber where it says:
"Node" should be a link to https://docs.ros.org/en/jazzy/p/rclpy/api/node.html
"String" should be a link to http://docs.ros.org/en/jazzy/p/std_msgs/interfaces/msg/String.html
And the links should be aligned with the ros version of the docs.
I believe this would help developers find and navigate the documentation and help with search engine optimization (as search engines "crawl" assigning values to linked-to sites).
The text was updated successfully, but these errors were encountered: