-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IsNumber matcher #207
Comments
@andreoss I see some commits associated to this ticket, but no PR, did you forget to create it? |
@victornoel It was merged already #208 |
@andreoss ah perfect, next time can you close or ping the bug reporter (or ARC) for the issue to be closed once an issue is fixed? :) |
@andreoss also don't hesitate to create an issue in cactoos for starting using this new matcher in the tests then! |
@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
@victornoel please make sure problem reported and problem solvers are 2 different persons |
@sereshqua yes, we are a bit short in active DEVs but nevertheless, I will try do to so in the future :) |
@0crat quality acceptable |
There are several classes in cactoos that extend
Number
(LengthOf
,DivisionOf
,AvgOf
,SumOf
,NumberOf
,NumberEnvelope
,MultiplicationOf
,MaxOf
,MinOf
,Sealed
).Let's introduce
IsNumber
in order to test them with avoiding callingintValue()
,longValue()
, etc on these objects.The text was updated successfully, but these errors were encountered: