Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Jul 11, 2020

@meshcollider
Copy link
Contributor

meshcollider commented Jul 11, 2020

Concept ACK

@practicalswift
Copy link
Contributor

ACK facd7dd

Getting rid of tpyos once in a while is good project hygiene :)

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK facd7dd

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK facd7dd, spelling verified with test/lint/lint-spelling.sh: all remaining warnings are false positive.

Copy link
Contributor

@meshcollider meshcollider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review & functional test run ACK facd7dd

@meshcollider meshcollider merged commit 32302e5 into bitcoin:master Jul 12, 2020
@maflcko maflcko deleted the 2007-docWallet branch July 12, 2020 05:10
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 12, 2020
facd7dd wallet: Fix typo in comments; Simplify assert (MarcoFalke)

Pull request description:

  Follow up to bitcoin#19046 (comment) and bitcoin#19046 (comment)

ACKs for top commit:
  practicalswift:
    ACK facd7dd
  jonatack:
    ACK facd7dd
  hebasto:
    ACK facd7dd, spelling verified with `test/lint/lint-spelling.sh`: all remaining warnings are false positive.

Tree-SHA512: 2b185d138058840db56726bb6bcc42e5288a954e2a410c49e04806a047fbbdaf0bb2decc70ecf7613c69caa766655705ca44151613e7ea5015b386d1e726d870
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Aug 31, 2021
Summary: This is a backport of [[bitcoin/bitcoin#19490 | core#19490]]

Test Plan: `ninja all check-all`

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D9992
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants