Skip to content

doc: Add post branch-off note about fuzz input pruning#27574

Merged
fanquake merged 1 commit into
bitcoin:masterfrom
dergoegge:2023-05-release-qa-prune
May 5, 2023
Merged

doc: Add post branch-off note about fuzz input pruning#27574
fanquake merged 1 commit into
bitcoin:masterfrom
dergoegge:2023-05-release-qa-prune

Conversation

@dergoegge
Copy link
Copy Markdown
Member

@dergoegge dergoegge commented May 4, 2023

No description provided.

@DrahtBot
Copy link
Copy Markdown
Contributor

DrahtBot commented May 4, 2023 •
edited
Loading

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
Concept ACK pablomartin4btc

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

@DrahtBot DrahtBot added the Docs label May 4, 2023
@dergoegge
Copy link
Copy Markdown
Member Author

dergoegge commented May 4, 2023

cc @MarcoFalke

maflcko
maflcko approved these changes May 5, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

lgtm

pablomartin4btc
pablomartin4btc approved these changes May 5, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@pablomartin4btc pablomartin4btc left a comment

Choose a reason for hiding this comment

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

ACK.
Btw, there's a typo on line 29 on that readme file: "corpora", since you are there also on line 32 perhaps should say "running" instead of "run".

@fanquake
Copy link
Copy Markdown
Member

fanquake commented May 5, 2023

"corpora",

This isn't a typo. It's a commonly used term when talking about fuzzing.

@fanquake fanquake merged commit 5d1014d into bitcoin:master May 5, 2023
@dergoegge
Copy link
Copy Markdown
Member Author

dergoegge commented May 5, 2023

@fanquake It is actually a typo, in the readme it is currently spelled "copora" not "corpora"

@fanquake
Copy link
Copy Markdown
Member

fanquake commented May 5, 2023

Right. Confusion over posting the correct spelling and calling it a typo.

@fanquake fanquake mentioned this pull request May 5, 2023
Merged
@fanquake
Copy link
Copy Markdown
Member

fanquake commented May 5, 2023

Fixed in bitcoin-core/qa-assets#123.

fanquake added a commit to bitcoin-core/qa-assets that referenced this pull request May 5, 2023
@fanquake
Merge #123: doc: fix typos
47963ef 
a7bbd1e doc: fix typos (fanquake)

Pull request description:

  Pointed out in bitcoin/bitcoin#27574.

ACKs for top commit:
  dergoegge:
    ACK a7bbd1e

Tree-SHA512: 1c07848e641d6da58cd96eb52d188d12d00f4cabc721b020bc0a7ecd3e210ae9d0ededbb36d06e20de08d0ebd9b1899cb488059e07d3f1aa9a62ec7c38867824
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 5, 2023
@fanquake
Merge bitcoin#27574: doc: Add post branch-off note about fuzz input p…
9610608 
…runing
@bitcoin bitcoin locked and limited conversation to collaborators May 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@maflcko maflcko maflcko approved these changes

@pablomartin4btc pablomartin4btc pablomartin4btc approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dergoegge @DrahtBot @fanquake @maflcko @pablomartin4btc