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 on IRC.

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 (although that might be a nice side-effect).

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?  Bitcoin Core contributor jnewbery started the review club and schedules the upcoming meetings. Individual meetings are hosted by a variety of Bitcoin Core contributors. See some of our previous hosts.

Upcoming Meetings

» #22871 Discourage CSV as NOP when locktime disable is set & discourage unknown nSequence (consensus) hosted by glozow

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

Recent Meetings

» #22677 Cut the validation <-> txmempool circular dependency (p2p, validation, mempool) hosted by glozow
» #22009 Decide which coin selection solution to use based on waste metric (wallet) hosted by glozow
» #22675 policy: extract RBF logic into policy/rbf (refactoring, tx fees and policy, validation) hosted by glozow
» #22665 Don't return incorrect replaceability status (tx fees and policy, rpc/rest/zmq) hosted by jnewbery

See all meetings.

Other Resources for New Contributors