From: merge-script Date: Sat, 7 Feb 2026 10:29:16 +0000 (+0000) Subject: Merge bitcoindevkit/bdk#2083: fix: no `Debug` on `Display` implementations X-Git-Url: http://internal-gitweb-vhost/%22https:/parse/scripts/rpc/io/struct.Error.html?a=commitdiff_plain;h=HEAD;p=bdk Merge bitcoindevkit/bdk#2083: fix: no `Debug` on `Display` implementations 823e4e980af03672cc6342d0e4850febcbd14c9d fix(file_store)!: remove Debug usage and implement Display for StoreError. (Dmenec) 5a85f6e25a9e14204cb66fec3f7381a59133f0be fix(chain)!: remove Debug usage and implement Display for CalculateFeeError (Dmenec) d0889f37ae1176eb1510a8d260cde8e05fe68f76 docs(core): add detailed SyncRequest output example (Dmenec) 5d715c2e4c8076be058e342efe279a3fc3481312 fix(chain)!: remove Debug usage and implement Display for InsertDescriptorError (Dmenec) Pull request description: ### Description Fixes https://github.com/bitcoindevkit/bdk/issues/1933, remove the usage of `Debug` on `Display` implemetations mentioned in https://github.com/bitcoindevkit/bdk/pull/1881#issuecomment-2718412512 ### Changelog notice - Replaced `Debug` usage with `Display` messages for `InsertDescriptorError`, `CalculateFeeError` and `StoreError`. - `InvalidMagicBytes` now displays expected and actual bytes in hexadecimal instead of using `Debug`. - Added a new `short_descriptor` function to shorten descriptors for concise error output. - Added detailed `SyncRequest` `Display` for Esplora and Electrum examples. ### Checklists #### All Submissions: * [x] I have signed all my commits * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md) * [x] I ran `just check`, `just fmt` and `just test` before committing ACKs for top commit: evanlinjin: ACK 823e4e980af03672cc6342d0e4850febcbd14c9d Tree-SHA512: fc3d66b72dc3f7ed3dee3b017687ece73c93883d23cbb019c402e55957368a6487e02b400b092169c326a47a86403f7aa865fe7d7238ccc6eb77afef258ec1ca --- b45ecb9bd41c80a03921dcf42a9845b19774502e