Skip to content

Implement signed modulo#162

Merged
mborland merged 4 commits intodevelopfrom
129
Mar 30, 2026
Merged

Implement signed modulo#162
mborland merged 4 commits intodevelopfrom
129

Conversation

@mborland
Copy link
Copy Markdown
Member

Closes: #129

@mborland mborland added this to the v0.1.0 milestone Mar 30, 2026
@mborland mborland self-assigned this Mar 30, 2026
@mborland mborland added the enhancement New feature or request label Mar 30, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 90.72848% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.8%. Comparing base (b5a1a28) to head (4e21abd).
⚠️ Report is 5 commits behind head on develop.

Files with missing lines Patch % Lines
...boost/safe_numbers/detail/signed_integer_basis.hpp 75.0% 10 Missing ⚠️
test/test_signed_modulo.cpp 96.4% 4 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop    #162     +/-   ##
=========================================
- Coverage     96.9%   96.8%   -0.1%     
=========================================
  Files          119     120      +1     
  Lines         9109    9260    +151     
  Branches      3446    3507     +61     
=========================================
+ Hits          8825    8962    +137     
- Misses         218     232     +14     
  Partials        66      66             
Files with missing lines Coverage Δ
test/test_signed_modulo.cpp 96.4% <96.4%> (ø)
...boost/safe_numbers/detail/signed_integer_basis.hpp 72.7% <75.0%> (+0.3%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5a1a28...4e21abd. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mborland mborland merged commit e79cd5e into develop Mar 30, 2026
42 checks passed
@mborland mborland deleted the 129 branch March 30, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement basic signed modulo

1 participant