A weekly review club for Bitcoin Core PRs

What is this?  A weekly club for reviewing Bitcoin Core PRs at 17:00 UTC on Wednesdays in the #bitcoin-core-pr-reviews IRC channel on libera.chat.

What’s it for?  To help newer contributors learn about the Bitcoin Core review process. The review club is not primarily intended to help open PRs get merged.

Who should take part?  Anyone who wants to learn about contributing to Bitcoin Core. All are welcome to come and ask questions!

What’s the benefit for participants?  Reviewing and testing PRs is the best way to start contributing to Bitcoin Core, but it’s difficult to know where to start. There are hundreds of open PRs, many require a lot of contextual knowledge, and contributors and reviewers often use unfamiliar terminology. The review club will give you the tools and knowledge you need in order to take part in the Bitcoin Core review process on GitHub.

How do I take part? Just show up on IRC! See Attending your first PR Review Club for more tips on how to participate.

Who runs this?  Upcoming meetings are scheduled by Bitcoin Core contributor glozow. The meetings are hosted by a variety of Bitcoin Core contributors. See some of our previous hosts.

Upcoming Meetings

» #24832 Verify the block filter hash when reading the filter from disk. (utxo db and indexes) hosted by stickies-v
» #25077 Fix chain tip data race and corrupt rest response (validation) hosted by MarcoFalke
» #25380 Detect and ignore transactions that were CPFP'd in the fee estimator (mempool) hosted by glozow
» #25218 Introduce generic 'Result' class and connect it to CreateTransaction and GetNewDestination (refactoring) hosted by stickies-v

We’re always looking for interesting PRs to discuss in the review club and for volunteer hosts to lead the discussion:

Recent Meetings

» #23418 Fix signed integer overflow in prioritisetransaction RPC (rpc/rest/zmq, mempool) hosted by MarcoFalke
» #24170 Manual block-relay-only connections with addnode (rpc/rest/zmq, p2p) hosted by mzumsande
» #22778 net processing: Reduce resource usage for inbound block-relay-only connections (p2p) hosted by larryruane
» #25228 Add BIP-125 rule 5 testcase with default mempool (tests) hosted by glozow

See all meetings.

Other Resources for New Contributors