Switch from create-hash, ripemd160 to noble-hashes

[paulmillr]

hash160, hash256 methods were adjusted because it's more performant to not do Buffer.from twice. I suppose some future update of bitcoinjs will remove buffer altogether.

create-hash and ripemd160 were not removed from dependency tree. Reasons:

• bip32: needs a new release
• bip39, bs58check: need switch to hashes

[junderw]

Maybe we should do bip39 and bs58check first, then cut releases for those and then do this.

[paulmillr]

bitcoinjs/bip39#188, bitcoinjs/bs58check#41

[paulmillr]

What do you think regarding merging those?

[junderw]

Would you mind converting the travis yml to a Github CI yml so that it can run the tests on the PR?

Thanks.

(Edit: referring to the 2 PRs you linked)

[paulmillr]

Done. Copy-pasted your ci config from the repository and adjusted a bit. All tests are passing.

[junderw]

I have merged and published everything.

Please remove create-hash 🎉

[junderw]

I suppose some future update of bitcoinjs will remove buffer altogether.

Maybe ChatGPT can help with this. lol

[paulmillr]

That did not help. wif still depends on old stuff.

bitcoinjs/wif#30

regtest-client also does, but it's devDependency - probably not worth to upgrade; users won't see create-hash after bitcoinjs upgrade already.

[junderw]

Is wif even used outside tests?

[paulmillr]

Not used anywhere, actually. Removed, all good now. create-hash became devDependency.

[paulmillr]

Seems good to go!

[paulmillr]

@junderw ping