Skip to content

Conversation

@LeonardEyer
Copy link
Contributor

Hi! First time contributing to open source. Let me know if I missed something.

I believe this small fix was overlooked in a74fc3b.

@codecov
Copy link

codecov bot commented Nov 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.20%. Comparing base (d517d20) to head (ed55543).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #276      +/-   ##
==========================================
+ Coverage   86.18%   86.20%   +0.02%     
==========================================
  Files           1        1              
  Lines         666      667       +1     
  Branches      124      125       +1     
==========================================
+ Hits          574      575       +1     
  Misses         92       92              
Files with missing lines Coverage Δ
include/nanoflann.hpp 86.20% <100.00%> (+0.02%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlblancoc
Copy link
Owner

oh, thanks, I think you are right! 👏

Great contribution, including the unit test 👍

@jlblancoc jlblancoc merged commit d8bfec8 into jlblancoc:master Nov 27, 2025
9 checks passed
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