Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix bin search error #22

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Fix bin search error #22

merged 1 commit into from
Aug 23, 2024

Conversation

Micki-D
Copy link
Collaborator

@Micki-D Micki-D commented Aug 22, 2024

Fixes the error found in issue #17

Copy link

codecov bot commented Aug 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.03%. Comparing base (84cac09) to head (e0a9aa5).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   97.03%   97.03%           
=======================================
  Files           7        7           
  Lines         405      405           
=======================================
  Hits          393      393           
  Misses         12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@oschulz
Copy link
Member

oschulz commented Aug 23, 2024

Thanks!

@oschulz oschulz merged commit edbf33f into bat:main Aug 23, 2024
10 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants