Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Sep 2, 2021

  • Remove unused yq
  • Update fuzzing docs

@fanquake fanquake added the Tests label Sep 2, 2021
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK fa050bb

To normalize the name of all three test runners (fuzz, functional, util).
@maflcko
Copy link
Member Author

maflcko commented Sep 2, 2021

Pushed another commit to clarify what test/config.ini.in is used for.

@Saviour1001
Copy link

ACK fa0937d

@DrahtBot
Copy link
Contributor

DrahtBot commented Sep 2, 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:

  • #21551 (ci: Move Windows MSVC build from AppVeyor to Cirrus by hebasto)

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.

@practicalswift
Copy link
Contributor

cr ACK fa0937d

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK fa0937d

@fanquake fanquake merged commit 78d39f8 into bitcoin:master Sep 6, 2021
@maflcko maflcko deleted the 2109-testDoc branch September 6, 2021 07:22
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 7, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 6, 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