scgbckbone
scgbckbone
> Please rebase on top of current master to remove the merge commit. will do squash and rebase after this is reviewed and ready to go - I assume this...
Few notes here. We use password field for CVC, which is something like pin code but you have to provide it for every auth command. Therefore it is not suitable...
rebased on current master
https://github.com/coinkite/coinkite-tap-proto/pull/24 merged
updated `poetry.lock` with `poetry update && poetry lock` after addition of `coinkite-tap-protocol` --> without this update installing via poetry would produce solver error
@achow101 rebased and fixed - will squash those commits after - I think it is easier for you to review changes unsquashed
`coldcard-multisig.patch` was outdated - fixed in https://github.com/bitcoin-core/HWI/pull/598/commits/38be6be5a9151d940badb20b90a60cc677c428f2
not sure about CI --> seems to be full of random failures and timeouts
[52aa685f95b01a9480c0fb07acb7e433dc98ef2a](https://github.com/bitcoin-core/HWI/commit/52aa685f95b01a9480c0fb07acb7e433dc98ef2a) Moved Tapsigner and its dependencies to extras. This aims to partly alleviate your concerns with regards to `sdist` and `wheel` distributions. In sdist and wheel, only references to dependencies...
what do you think @SomberNight ? I'd say it is ready...