Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Fix maximum amount function - Closes #122 #131

Merged
merged 2 commits into from
Apr 24, 2017
Merged

Fix maximum amount function - Closes #122 #131

merged 2 commits into from
Apr 24, 2017

Conversation

slaweet
Copy link
Contributor

@slaweet slaweet commented Apr 20, 2017 •
edited by karmacoma
Loading

By avoiding floating point arithmetic.

Closes #122

@slaweet
Fix maximum amount function #122
fc5294d 
by avoiding floating point arithmetics
@slaweet slaweet requested a review from karmacoma April 20, 2017 12:30
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 82.372% when pulling fc5294d on 122_maximum_amount_invalid into 3b74f84 on 0.2.1.

@karmacoma karmacoma changed the title Fix maximum amount function #122 Fix maximum amount function - Closes #122 Apr 20, 2017
@karmacoma karmacoma added the bug label Apr 20, 2017
@karmacoma karmacoma self-assigned this Apr 21, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 82.372% when pulling f3747fb on 122_maximum_amount_invalid into f6b03d2 on 0.2.1.

@karmacoma karmacoma merged commit e9b0f21 into 0.2.1 Apr 24, 2017
@karmacoma karmacoma deleted the 122_maximum_amount_invalid branch April 24, 2017 14:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@karmacoma karmacoma karmacoma approved these changes

Assignees

@karmacoma karmacoma

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@slaweet @coveralls @karmacoma