no2chem / bigint-buffer

💪🔢 bigint-buffer: Buffer Utilities for TC39 BigInt Proposal
Apache License 2.0
54 stars 11 forks source link

Failed to load bindings, pure JS will be used #31

Open mytecor opened 3 years ago

mytecor commented 3 years ago

1.1.5 on windows

gnarea commented 1 year ago

According to https://github.com/trufflesuite/ganache/issues/1080#issuecomment-906550239, this warning can be avoided by having the build tools pre-installed.

jmrossy commented 11 months ago

Same on MacOS

jmrossy commented 11 months ago

GIven that fallback to pure JS is possible, it seems like this line just creates noise and confusion: https://github.com/no2chem/bigint-buffer/blob/master/src/index.ts#L16

I'd recommend simply removing it. It's better if libs like this don't log to console. If you really want to log anyway, you can use a lib like debug and log under your namespace, so devs can optionally enable that logging as desired.