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

Remove unneeded boost bind include #702

Merged

Conversation

jdlangs
Copy link
Contributor

@jdlangs jdlangs commented Jan 24, 2022

Not needed since C++11 and this header puts placeholder objects in the global namespace on system-installed Boost versions

Not needed since C++11 and this header puts placeholder objects in the
global namespace on system-installed Boost versions
@Levi-Armstrong
Copy link
Contributor

I will merge once CI passes.

@codecov
Copy link

codecov bot commented Jan 24, 2022

Codecov Report

Merging #702 (b0a2a25) into master (eade7f1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #702   +/-   ##
=======================================
  Coverage   92.16%   92.16%           
=======================================
  Files         173      173           
  Lines       11097    11097           
=======================================
  Hits        10227    10227           
  Misses        870      870           
Impacted Files Coverage Δ
...sion/core/include/tesseract_collision/core/types.h 100.00% <ø> (ø)

@Levi-Armstrong Levi-Armstrong merged commit 18da3f1 into tesseract-robotics:master Jan 25, 2022
@jdlangs jdlangs deleted the fix/boost-bind-include branch January 25, 2022 16:58
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.

2 participants