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

Doxygen tag #1955

Merged
merged 3 commits into from
Feb 17, 2023
Merged

Doxygen tag #1955

merged 3 commits into from
Feb 17, 2023

Conversation

henningkayser
Copy link
Member

I'm working on fixing the dead links in the tutorials, and making it easier to reference the API using doxylink. We need to install a tag file for that to work. Also, the QT property is breaking the references because of the identical naming, with the attached fixed everything works fine.

@henningkayser henningkayser added backport-foxy Mergify label that triggers a PR backport to Foxy backport-galactic Mergify label that triggers a PR backport to Galactic backport-humble Mergify label that triggers a PR backport to Humble labels Feb 17, 2023
Copy link
Member

@tylerjw tylerjw left a comment

Choose a reason for hiding this comment

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

If this passes, lets merge this and backport to humble?

@tylerjw
Copy link
Member

tylerjw commented Feb 17, 2023

  --- stderr: moveit_msgs
  CMake Error at /opt/ros/rolling/share/sensor_msgs/cmake/ament_cmake_export_dependencies-extras.cmake:21 (find_package):
    By not providing "Findservice_msgs.cmake" in CMAKE_MODULE_PATH this project
    has asked CMake to find a package configuration file provided by
    "service_msgs", but CMake did not find one.
  
    Could not find a package configuration file provided by "service_msgs" with
    any of the following names:
  
      service_msgsConfig.cmake
      service_msgs-config.cmake

is there something broken about moveit_msgs now?

@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Base: 50.29% // Head: 50.30% // Increases project coverage by +0.02% 🎉

Coverage data is based on head (5b545f9) compared to base (b1b554e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1955      +/-   ##
==========================================
+ Coverage   50.29%   50.30%   +0.02%     
==========================================
  Files         374      374              
  Lines       31358    31358              
==========================================
+ Hits        15769    15773       +4     
+ Misses      15589    15585       -4     
Impacted Files Coverage Δ
...e/collision_detection_fcl/src/collision_common.cpp 73.99% <0.00%> (+0.23%) ⬆️
...dl_kinematics_plugin/src/kdl_kinematics_plugin.cpp 79.17% <0.00%> (+1.14%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tylerjw
Copy link
Member

tylerjw commented Feb 17, 2023

I'm going to merge this as the failure has nothing to do with this PR and should be addressed separately.

@tylerjw tylerjw merged commit 752571e into moveit:main Feb 17, 2023
mergify bot pushed a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)
mergify bot pushed a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)
mergify bot pushed a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)
henningkayser added a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)

Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
henningkayser added a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)

Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
henningkayser added a commit that referenced this pull request Feb 17, 2023
* Generate Doxygen Tag

* Install tagfile in output directory

* Fix problematic override for Doxygen linking

(cherry picked from commit 752571e)

Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-foxy Mergify label that triggers a PR backport to Foxy backport-galactic Mergify label that triggers a PR backport to Galactic backport-humble Mergify label that triggers a PR backport to Humble
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants