Skip to content

Fix outdated float64 doc comments in helper package#353

Merged
cinar merged 2 commits intocinar:masterfrom
whencanibe:fix-outdated-float64-comments-in-helper
Apr 19, 2026
Merged

Fix outdated float64 doc comments in helper package#353
cinar merged 2 commits intocinar:masterfrom
whencanibe:fix-outdated-float64-comments-in-helper

Conversation

@whencanibe
Copy link
Copy Markdown
Contributor

Describe Request

Doc comments in the helper package still referenced float64 despite the functions having been migrated to generics. This PR updates the comments in all 20 affected files to accurately reflect the generic type parameters (T, F) used in their signatures.

Change Type

Documentation fix (no functional changes)

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.13%. Comparing base (e86d913) to head (6d1fb8c).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #353   +/-   ##
=======================================
  Coverage   93.13%   93.13%           
=======================================
  Files         215      215           
  Lines        6190     6190           
=======================================
  Hits         5765     5765           
  Misses        344      344           
  Partials       81       81           

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

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

@cinar cinar merged commit f5bf394 into cinar:master Apr 19, 2026
2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 19, 2026
@whencanibe whencanibe deleted the fix-outdated-float64-comments-in-helper branch April 22, 2026 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants