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 missing <functional> header #382

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Conversation

mjcarroll
Copy link
Contributor

Found when building from source on 22.04 with clang-14.

Signed-off-by: Michael Carroll <michael@openrobotics.org>
@github-actions github-actions bot added the 🎵 harmonic Gazebo Harmonic label Sep 12, 2023
@azeey azeey added the beta Targeting beta release of upcoming collection label Sep 12, 2023
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Merging #382 (7165577) into gz-msgs10 (50ee1e4) will not change coverage.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           gz-msgs10     #382   +/-   ##
==========================================
  Coverage      97.26%   97.26%           
==========================================
  Files             27       27           
  Lines           1169     1169           
==========================================
  Hits            1137     1137           
  Misses            32       32           
Files Changed Coverage Δ
core/include/gz/msgs/PointCloudPackedUtils.hh 94.91% <ø> (ø)

@mjcarroll mjcarroll merged commit 74dc3b4 into gz-msgs10 Sep 12, 2023
10 of 12 checks passed
@mjcarroll mjcarroll deleted the mjcarroll/missing_headers branch September 12, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Targeting beta release of upcoming collection 🎵 harmonic Gazebo Harmonic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants