Skip to content

Conversation

@Sjors
Copy link
Member

@Sjors Sjors commented Mar 1, 2023

Followups for #26032. So far nothing major.

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 1, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK ishaanam, S3RK

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@Sjors Sjors force-pushed the 2023/03/pr26032-followups branch from 71c0bd2 to 6fc5f4f Compare March 1, 2023 11:45
@fanquake fanquake changed the title 26032 followups wallet: 26032 followups Mar 1, 2023
@DrahtBot DrahtBot added the Wallet label Mar 1, 2023
@fanquake
Copy link
Member

fanquake commented Mar 7, 2023

Followups for #26032. So far nothing major.

Are you expecting there to be more? I really only see the one followup comment in #26032.

@Sjors Sjors marked this pull request as ready for review March 7, 2023 16:50
@Sjors
Copy link
Member Author

Sjors commented Mar 7, 2023

Not really I guess...

@Sjors Sjors changed the title wallet: 26032 followups doc: DummySignInput mention external signer Mar 7, 2023
@fanquake
Copy link
Member

fanquake commented Mar 7, 2023

cc @S3RK @ishaanam @furszy

@ishaanam
Copy link
Contributor

ishaanam commented Mar 7, 2023

ACK 6fc5f4f

@S3RK
Copy link
Contributor

S3RK commented Mar 8, 2023

ACK 6fc5f4f

The comment is correct, CI failure is unrelated

@fanquake fanquake merged commit 69ba572 into bitcoin:master Mar 8, 2023
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 8, 2023
6fc5f4f doc: DummySignInput mention external signer (Sjors Provoost)

Pull request description:

  Followups for bitcoin#26032. So far nothing major.

ACKs for top commit:
  ishaanam:
    ACK 6fc5f4f
  S3RK:
    ACK 6fc5f4f

Tree-SHA512: e27edde9853487fe3eef8213f991aae3724f318bbbe0b11da23759879adaf9a31771e6ea0c30baaebca149032780b89b32aa540ff456ca3d5ec6adb0371749c6
@bitcoin bitcoin locked and limited conversation to collaborators Mar 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants