replacing bn.js

[dynst]

cosmjs/packages/math/src/decimal.ts

Lines 108 to 111 in 318c8d5

	private readonly data: {
	readonly atomics: BN;
	readonly fractionalDigits: number;
	};

This could just use BigInt now, right? It's been available since Node.js 10.4.

[webmaster128]

Last time I looked into this I was blocked by a good bunch of mathematical operations which are available in bn.js but not in BigInt. Not sure what that was.

We could make such replacements one by one but I assume we still have bn.js in the dependency tree anyways.

[dynst]

but I assume we still have bn.js in the dependency tree anyways.

That was true until you removed elliptic. #1272

yarn why bn.js says these are the only 2 users left now.

├─ @cosmjs/crypto@workspace:packages/crypto
│  └─ bn.js@npm:5.2.0 (via npm:^5.2.0)
│
└─ @cosmjs/math@workspace:packages/math
   └─ bn.js@npm:5.2.0 (via npm:^5.2.0)