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

Add custom rosdoc2 config for ros2_control metapackage #1484

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

christophfroehlich
Copy link
Contributor

@christophfroehlich christophfroehlich commented Apr 16, 2024

rosdoc2 will be used by the build farm to publish API docs on http://docs.ros.org/en/rolling/p/ros2_control/ which is empty now.

image

@christophfroehlich christophfroehlich added backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron. labels Apr 16, 2024
Copy link

codecov bot commented Apr 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.67%. Comparing base (16fbde3) to head (2559763).
Report is 3 commits behind head on master.

❗ Current head 2559763 differs from pull request most recent head bad52a3. Consider uploading reports for the commit bad52a3 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1484   +/-   ##
=======================================
  Coverage   62.67%   62.67%           
=======================================
  Files          96       96           
  Lines       11311    11311           
  Branches     8151     8151           
=======================================
  Hits         7089     7089           
  Misses        715      715           
  Partials     3507     3507           
Flag Coverage Δ
unittests 62.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

- `API <http://docs.ros.org/en/rolling/p/ros2_control_test_assets/>`__
- `ROS Index <https://index.ros.org/p/ros2_control_test_assets/#rolling>`__
* - transmission_interface
- `API <http://docs.ros.org/en/rolling/p/transmission_interface/>`__
Copy link
Member

Choose a reason for hiding this comment

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

@christophfroehlich sorry for not writing earlier

I see it's missing ros2controlcli, rqt_controller_manager etc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is this really helpful? I'm not sure, this is not meant to be of public API?
http://docs.ros.org/en/rolling/p/ros2controlcli/
http://docs.ros.org/en/rolling/p/rqt_controller_manager/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble This label should be used by maintaines only! Label triggers PR backport to ROS2 humble. backport-iron This label should be used by maintaines only! Label triggers PR backport to ROS2 Iron.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants