Remove bignum (because it's outdated and a mess)

zone117x / node-cryptonote-pool

Mining pool for CryptoNote based coins such as Bytecoin and Monero

GNU General Public License v2.0

823 stars 1.14k forks source link

Remove bignum (because it's outdated and a mess) #380

Closed OlMi1 closed 2 years ago

OlMi1 commented 2 years ago

Removed reference to bignum! This means that we can save ourselves from installing the mess that bignum is! On a technical level, bignum is outdated and the developers want you to use JS's BigInt. The given changes should change it to BigInt.

Since I am not a professional developer, I would kindly ask for people to check my code before merging it in and review issues that may happen. I made sure to check all occurences of the only bignum in the project, diff1.

OlMi1 commented 2 years ago

I noticed a few errors. Got some work left to do!