Skip to content

Ensure all headers get installed within moveit_core directory#1786

Merged
AndyZe merged 1 commit intomoveit:mainfrom
ChrisThrasher:include_prefix
Dec 6, 2022
Merged

Ensure all headers get installed within moveit_core directory#1786
AndyZe merged 1 commit intomoveit:mainfrom
ChrisThrasher:include_prefix

Conversation

@ChrisThrasher
Copy link
Copy Markdown
Contributor

Description

Fixes an oversight from #1785 so that now all moveit_core headers reside in this new moveit_core header install directory.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 6, 2022

Codecov Report

Base: 50.24% // Head: 50.24% // No change to project coverage 👍

Coverage data is based on head (6178b24) compared to base (7c79812).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1786   +/-   ##
=======================================
  Coverage   50.24%   50.24%           
=======================================
  Files         374      374           
  Lines       31277    31277           
=======================================
  Hits        15711    15711           
  Misses      15566    15566           

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.

@AndyZe AndyZe merged commit 939b678 into moveit:main Dec 6, 2022
@ChrisThrasher ChrisThrasher deleted the include_prefix branch December 6, 2022 21:13
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.

3 participants