Elias Rohrer

> Height + Hash would be better. I'm not too familiar with the bdk internals so I suggested unix time since you can immediately calculate that and store it on...

> We are now using the `CombindedChangeSet` type for persisting `Wallet` data and that type implements `Default`, so all fields are optional or can be empty. I think this lets...

Recently stumbled across this again when implementing Electrum syncing for LDK. Would be great to get a resolution here: is `proxy` mandatory, if so, why is it a feature to...

Hmm, given this has been used in production for a while by LSPS2 just-in-time channel providers, should we move forward with merging this eventually? (cc @valentinewallace @TheBlueMatt @t-bast)

> Hmm so it sounds like this bLIP is not very compatible with splicing and may be deprecated in the near future? I don't feel strongly about what to do...

I'm +1 for re-adding this functionality. Previously, I used the `TransactionDetails` returned from `TxBuilder::finish` to run some checks (sent/received amount, adherence to a fee limit, etc) *before* proceeding to sign...

> We don't have to test on MSRV we just have to check that it builds on MSRV. dev-dependencies shouldn't have to be pinned. I disagree: MSRV means "This project...

> This "regularly" is quite concerning tbh 😅. It's a good point that if our tests don't have the same MSRV as build then there's no way to verify that...

I'm afraid that's not easily possible, since it's part of a larger simulation framework. Due to this fact, I also cannot pinpoint the error to a specific connect call, but...

The nodes are part of a topology of routers and leaf nodes. As discussed in https://github.com/signetlabdei/quic-ns-3/issues/12, I now install the QUIC stack when needed: ``` if (enableQuic) { ns3::QuicHelper stack;...