[go: up one dir, main page]

MIR: switch to BigInt/BigUint

Context

This has been a long-standing issue that we're using i128/u128 instead of bigints. We kept putting this off due to some uncertainty about which bigint implementation to choose.

We need bigints for bls arithmetic however, so about time we chose something, I reckon.

I went with num-bigint to simplify interop with the Kernel SDK.

Resolves #6410 (closed)

Based on: !10972 (merged)

Manually testing the MR

cargo test

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Nikolay Yakimov

Merge request reports

Loading