Skip to content
This repository has been archived by the owner on Jan 2, 2020. It is now read-only.

Add BigDecimal compatibility for ruby 2.6 #13

Merged
merged 1 commit into from
Dec 11, 2018
Merged

Conversation

bobmaerten
Copy link
Collaborator

Filter string to pass on to Bigdecimal in to_multiplier method to
prevent errors like:

ArgumentError: invalid value for BigDecimal()

@bobmaerten bobmaerten requested a review from czj December 11, 2018 11:25
@czj
Copy link
Contributor

czj commented Dec 11, 2018

Only macOS fails with Ruby 2.6.0-rc1

@bobmaerten
Copy link
Collaborator Author

Tiens on se demandait pourquoi Travis fait 2 build sur chaque PR, mais en fait c'est écrit dans le détail. Pour une PR, il passe les tests sur la branche de la PR, et il simule un merge de la branche vers sa cible pour passer les tests dessus.

Malin.... Très malin !

Branch

This is a normal build for the prepare_for_ruby_2.6 branch. You should be able to reproduce it by checking out the branch locally.

PR

This is a pull request build.

It is running a build against the merge commit, after merging #13 Add BigDecimal compatibility for ruby 2.6.
Any changes that have been made to the master branch before the build ran are also included.

@bobmaerten
Copy link
Collaborator Author

bobmaerten commented Dec 11, 2018

Chez moi ça marche ™️ en ruby 2.6.0-rc1

~/Work/fuel_surcharge prepare_for_ruby_2.6*
❯ ruby -v
ruby 2.6.0rc1 (2018-12-06 trunk 66253) [x86_64-darwin18]

~/Work/fuel_surcharge prepare_for_ruby_2.6*
❯ rake
/Users/bob/.rbenv/versions/2.6.0-rc1/lib/ruby/gems/2.6.0/gems/addressable-2.5.2/lib/addressable/idna/pure.rb:154: warning: assigned but unused variable - startercc
Run options: --seed 46359

# Running:

..............

Finished in 4.450880s, 3.1454 runs/s, 15.9519 assertions/s.

14 runs, 71 assertions, 0 failures, 0 errors, 0 skips

@czj
Copy link
Contributor

czj commented Dec 11, 2018 via email

@czj
Copy link
Contributor

czj commented Dec 11, 2018 via email

@czj czj merged commit 4feb10a into master Dec 11, 2018
@czj czj deleted the prepare_for_ruby_2.6 branch December 11, 2018 21:53
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.

2 participants