Skip to content

Conversation

@Levi-Armstrong
Copy link
Contributor

No description provided.

@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch from 8f7142c to 3f64e65 Compare September 19, 2025 21:56
@codecov
Copy link

codecov bot commented Sep 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.04%. Comparing base (9742d52) to head (085ae66).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   98.41%   99.04%   +0.63%     
==========================================
  Files           2        3       +1     
  Lines         189      210      +21     
==========================================
+ Hits          186      208      +22     
+ Misses          3        2       -1     
Files with missing lines Coverage Δ
include/boost_plugin_loader/plugin_loader.h 100.00% <100.00%> (ø)
include/boost_plugin_loader/plugin_loader.hpp 100.00% <100.00%> (+0.90%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch from 3f64e65 to 6b10bac Compare September 20, 2025 18:41
@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch 2 times, most recently from 7cee73a to 2441780 Compare September 20, 2025 19:49
@Levi-Armstrong
Copy link
Contributor Author

@marip8 I fixed the CI issues and this is now ready to review.

@marip8
Copy link
Contributor

marip8 commented Sep 22, 2025

Is this supposed to replace #34?

@Levi-Armstrong
Copy link
Contributor Author

Is this supposed to replace #34?

I do not think so, I think it is still needed.

@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch from 2441780 to ca8547d Compare October 1, 2025 19:40
@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch 3 times, most recently from 1ab428d to 8bd9b52 Compare October 11, 2025 21:52
@Levi-Armstrong Levi-Armstrong force-pushed the feature/leverage-library-cache branch from 8bd9b52 to dc5a04a Compare October 11, 2025 22:21
@Levi-Armstrong
Copy link
Contributor Author

@marip8 This should be ready to review.

Copy link
Contributor

@marip8 marip8 left a comment

Choose a reason for hiding this comment

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

Just a few minor comments; looks good overall though

Co-authored-by: Michael Ripperger <michael.ripperger@swri.org>
@Levi-Armstrong Levi-Armstrong merged commit 750125d into tesseract-robotics:main Oct 14, 2025
9 checks passed
@Levi-Armstrong Levi-Armstrong deleted the feature/leverage-library-cache branch October 14, 2025 13:55
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