search

tezos-checker / checker

An in-development "robocoin" system for the Tezos blockchain
24 stars 16 forks source link

Represent tez as an Int64.t #192

Closed gkaracha closed 3 years ago

gkaracha commented 3 years ago

Fixes #190.

Many of our tests were dealing with very high amounts of tez, leading to failures after switching to Int64.t. So, I changed most generators for tez to go up to 10_000tez, to avoid hitting the Int64.max_int limit.

gkaracha commented 3 years ago

I think I would also personally feel better to have a second review (@utdemir) before merging given how core of a change this is.

Totally agree @dorranh, it's already uncomfortable enough changing something so fundamental so late, the more pairs of eyes check it the better!