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 parse(Decimal, "1e0") #59

Merged
merged 2 commits into from
Oct 14, 2024
Merged

fix parse(Decimal, "1e0") #59

merged 2 commits into from
Oct 14, 2024

Conversation

oscardssmith
Copy link
Member

Off by 1 in the branch for the exponent.

Off by 1 in the branch for the exponent.
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.21%. Comparing base (01a4884) to head (19ff2a3).
Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage   96.29%   99.21%   +2.92%     
==========================================
  Files           6        6              
  Lines         108      128      +20     
==========================================
+ Hits          104      127      +23     
+ Misses          4        1       -3     

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

Copy link

@fingolfin fingolfin left a comment

Choose a reason for hiding this comment

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

Looks good to me (but I am not a committer)

@tinybike tinybike merged commit d532c79 into master Oct 14, 2024
8 checks passed
@oscardssmith oscardssmith deleted the oscardssmith-patch-1 branch October 14, 2024 01:47
# 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.

3 participants