goyang icon indicating copy to clipboard operation
goyang copied to clipboard
verified publisher icon openconfig

The decimal's range should not be parsed if parsing fraction digits fails

Open fredgan opened this issue 5 years ago • 2 comments

fredgan avatar Nov 11 '20 02:11 fredgan

Coverage Status

Coverage increased (+0.3%) to 76.357% when pulling 18da630fa0d168a7b3cf5ef3a40cf3e501da4931 on fredgan:decimal-type-resolve-optimization into 88fe1d5cc9cee59fd3e95f42eddae19bb160bea5 on openconfig:master.

coveralls avatar Nov 11 '20 02:11 coveralls

I think this is reasonable to add, although I would want to wait until an integration test is added, which should be simple once #188 is merged. If that checks out then LGTM.

wenovus avatar Jul 21 '21 19:07 wenovus