Open stratospher opened 6 months ago
Session #1: 21 Dec 2023
Reviewed refactor: share and use GenerateRandomKey helper #28455 during the call.
Session #2: 4 Jan 2024
Reviewed test: Use test framework utils in functional tests #28528
during the call.
Left review comments:
Session #3: 25 Jan 2024
Reviewed test: p2p: check disconnect due to lack of desirable service flags #29279
during the call.
Left review comments:
Overview:
Session #4: 8 Feb 2024
Reviewed rpc: parse legacy pubkeys consistently with specific error messages #28336
during the call.
Left review comments:
Overview:
HexToPubKey
function
Reviewing PRs is the best way to contribute to Bitcoin Core. We pick a few easy PRs and review them together on the call. The aim of this session is only to show participants that reviewing PRs isn't intimidating.
Next Sessions
Session 5 on 22 Feb 2024. At Bitshala Disord.
We'll be reviewing PR 27114 which deals with whitelisting manually added peer connections.
Previous Sessions
PRs to review
How many PRs do you think we can cover in our 1 hour call? :) Feel free to add suggestions.
Reference
This is an excellent resource capturing the essence of reviewing - https://jonatack.github.io/articles/how-to-review-pull-requests-in-bitcoin-core!