Open mikemaccana opened 1 year ago
Looks like https://github.com/project-serum/serum-ts/blob/master/packages/borsh/src/index.ts still uses 'bn'.js' - is there a technical reason we wouldn't ditch the 'bn.js' dependency in favour of native BigInt?
BigInt works in most places since 2019 / 2020. Removing 'bn.js' would remove an unnecessary dependency and reduce the surface area for security issues and vulnerabilities.
Looks like https://github.com/project-serum/serum-ts/blob/master/packages/borsh/src/index.ts still uses 'bn'.js' - is there a technical reason we wouldn't ditch the 'bn.js' dependency in favour of native BigInt?
BigInt works in most places since 2019 / 2020. Removing 'bn.js' would remove an unnecessary dependency and reduce the surface area for security issues and vulnerabilities.