Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Jun 8, 2020

Remove an accidental leftover comment from #19082, which no longer applies and thus might be confusing

MarcoFalke added 2 commits June 8, 2020 08:10
Can be reviewed with the git options
--word-diff-regex=. --ignore-all-space  -U0
@DrahtBot DrahtBot added the Tests label Jun 8, 2020
@adamjonas
Copy link
Member

code review ACK fa98e10

@fanquake fanquake merged commit 45a6811 into bitcoin:master Jun 11, 2020
@fanquake
Copy link
Member

This addressed @gzhao408's comment from #19082. cc @jnewbery.

@maflcko maflcko deleted the 2006-testComment branch June 11, 2020 10:47
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jun 11, 2020
fa98e10 test: Remove leftover comment in mining_basic (MarcoFalke)
faedb50 test: pep-8 mining_basic (MarcoFalke)

Pull request description:

  Remove an accidental leftover comment from bitcoin#19082, which no longer applies and thus might be confusing

ACKs for top commit:
  adamjonas:
    code review ACK fa98e10

Tree-SHA512: c7f7f8f579b3c6e92f45769be0a7af1a421438a3f5524db5278b2269511a9e0e08f44e3836afb26727644035897ee51ff8296d13ce23030549e7403f57b40e40
@jnewbery
Copy link
Contributor

ACK. Thanks for picking this up, Marco.

Mitek75 pushed a commit to Mitek75/bitcoin that referenced this pull request Jun 14, 2020
…basic"

This reverts commit 45a6811, reversing
changes made to 6762a62.
Mitek75 pushed a commit to Mitek75/bitcoin that referenced this pull request Jun 14, 2020
@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.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants