A weekly review club for Bitcoin Core PRs

All are welcome!

We'll host a series of weekly review clubs on the #bitcoin-core-pr-reviews Freenode IRC channel at 17:00 UTC on Wednesdays. All are welcome to join and ask questions!

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. This weekly IRC club is for people who want to help review Bitcoin Core PRs but find the process intimidating. Everyone is welcome to speak and questions are encouraged!

To take part, you should:

The point of the review club is to give participants the tools and knowledge they need to take part in the Bitcoin Core review process on github. Inclusion of a PR in the review club is not an endorsement of the concept, approach or implementation in the PR!

Upcoming meetings

» #15931 Remove GetDepthInMainChain dependency on locked chain interface (wallet)
» #15759 Add 2 outbound blocks-only connections (p2p)

We're always looking for interesting PRs to discuss in review club and for volunteer hosts to lead the discussion. To suggest a PR or to offer to host a meeting, please leave a comment on this github issue.

Previous meetings

» #16115 On bitcoind startup, write config args to debug.log (config)
» #16345 Add getblockbyheight method / #16439 support @height in place of blockhash for getblock etc (rpc)
» #15505 [p2p] Request NOTFOUND transactions immediately from other outbound peers, when possible (p2p)
» #15713 refactor: Replace chain relayTransactions/submitMemoryPool by higher method (wallet)
» #15169 Parallelize CheckInputs() in AcceptToMemoryPool() (mempool)
» #16244 Move wallet creation out of the createwallet rpc into its own function (wallet)
» #15443 Add getdescriptorinfo functional test (tests)
» #15681 Allow one extra single-ancestor transaction per package (mempool)
» #15481 Restrict timestamp when mining a diff-adjustment block to prev-600 (mining)
» #15996 Deprecate totalfee argument in `bumpfee` (RPC)
» #16060 Bury bip9 deployments (consensus)
» #15741 Batch write imported stuff in importmulti (wallet)
» #15450 [GUI] Create wallet menu option (GUI)
» #15834 Fix NOTFOUND bug and expire getdata requests for transactions (net processing)
» #10823 Allow all mempool txs to be replaced after a configurable timeout (default 6h) (mempool/policy)
» #15557 Enhance bumpfee to include inputs when targeting a feerate (wallet)

Other resources for new contributors