Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Jun 4, 2021

This is confusing and might even slow down the test.

This reverts a change that was added a year ago in d438d60 and then the need for it was removed by 95d5d5e six months ago.

@fanquake fanquake added the Tests label Jun 4, 2021
@DrahtBot
Copy link
Contributor

DrahtBot commented Jun 4, 2021

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

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 346e52a this reverts a change that was added a year ago in d438d60 and then the need for it was removed by 95d5d5e six months ago

(perhaps add this context to the PR description)

@maflcko
Copy link
Member Author

maflcko commented Jun 5, 2021

Thanks for digging up the context! Added to description.

@maflcko maflcko merged commit 898dd9e into bitcoin:master Jun 5, 2021
@maflcko maflcko deleted the 2106-testFaster branch June 5, 2021 06:42
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jun 6, 2021
gwillen pushed a commit to ElementsProject/elements that referenced this pull request Jun 1, 2022
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 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.

4 participants