to_number accepts arguments of type json.Number#63
to_number accepts arguments of type json.Number#63arl wants to merge 1 commit intojmespath:masterfrom
Conversation
|
More context on the issue (and why the fix): |
|
I think this PR is reasonable and should be merged. @jamesls, what's your thought about it? |
|
@arl I’m working on expanding this implementation to bring it up on par to the latest version of JMESPath Community, I would be interested in taking this PR. Is there a specific compliance test that would show the failure before this PR gets merged ? |
Hi @springcomp yes there should be a test. |
to_numberimplementation fails to cast arguments of the standard typejson.Number.All numbers are converted to json.Number (a mere string) when
UseNumberhas been set on the json Decoder.