script: update REVIEWERS #21823

pull jonatack wants to merge 1 commits into bitcoin:master from jonatack:update-REVIEWERS changing 1 files +14 −4
  1. jonatack commented at 3:31 PM on May 1, 2021: contributor

    Meta: git show a06eb03 indicates the commit was first made one year ago and the PR was merged in September 2020.

    It might be time for an update, if automated review requests via DrahtBot are operational.

    "Regular contributors are free to add their names to specific directories or files provided that they are willing to provide a review."

    Perhaps we can compile and squash suggested updates here, per that guideline.

  2. jonatack commented at 3:32 PM on May 1, 2021: contributor

    Tagging @adamjonas added as author.

  3. fanquake added the label Docs on May 2, 2021
  4. laanwj commented at 12:56 PM on May 2, 2021: member

    ACK 5f4449bba1526d6630173a507b51766d7d7d382f Also adds @hebasto which is good. (but might be good to hold off immediate merging, as said in the OP, to aggregate more desired changes)

  5. Sjors commented at 9:25 AM on May 3, 2021: member

    Feel free to add below Descriptors:

    # External signer
*external_signer* [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)
docs/external-signer.md [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)
*signer.py [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)

    And somewhere else:

    # Backwards compatibility tests
test/functional/feature_backwards_compatibility.py [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)
test/functional/mempool_compatibility.py [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)
test/get_previous_releases.py [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/)
test/functional/wallet_upgradewallet.py [@sjors](/github-metadata-backup-bitcoin-bitcoin/contributor/sjors/) [@achow101](/github-metadata-backup-bitcoin-bitcoin/contributor/achow101/)

    (maybe use *_compatibility.py for the first two lines)

  6. MarcoFalke commented at 9:28 AM on May 3, 2021: member

    It wasn't clear to me whether people liked what DrahtBot does, so I am not running it 24/7. Since people are still happily adding themselves to the list, I presume I should run it more often?

  7. Sjors commented at 9:31 AM on May 3, 2021: member

    I don't mind getting a notification, though it might be annoying for copyright year changes and stuff like that.

  8. jonatack force-pushed on May 3, 2021
  9. jonatack commented at 9:40 AM on May 3, 2021: contributor

    @Sjors nice; updated.

  10. MarcoFalke commented at 9:41 AM on May 3, 2021: member

    annoying for copyright year changes

    Sure, but this change should only happen once a year per file. Though, it is currently unclear if the listed reviewers want to be notified about refactoring changes of the listed files.

  11. jonatack force-pushed on May 3, 2021
  12. jonatack commented at 9:50 AM on May 3, 2021: contributor

    Maybe sending review notifications a couple of times a week would be a good frequency, as a suggestion.

  13. jonatack cross-referenced this on May 3, 2021 from issue Update reviewers file: Remove too broad (`*.sh`) entry. Keep narrower entries. by practicalswift
  14. jonatack closed this on May 3, 2021
  15. jonatack commented at 11:02 AM on May 3, 2021: contributor

    Re-opening, sorry, crossed signals.

  16. jonatack reopened this on May 3, 2021
  17. script: update REVIEWERS 48bd9ebc5d
  18. jonatack force-pushed on May 3, 2021
  19. practicalswift commented at 1:10 PM on May 3, 2021: contributor

    ACK 48bd9ebc5db92af5162598d9a6d32a765252c701: patch looks correct :)

  20. hebasto approved
  21. hebasto commented at 4:03 PM on May 3, 2021: member

    ACK 48bd9ebc5db92af5162598d9a6d32a765252c701, I have reviewed the code and it looks OK, I agree it can be merged.

  22. adamjonas commented at 4:36 PM on May 3, 2021: member

    Glad to hear people still want this.

    ACK 48bd9eb. I ran through the new files added with *s using git ls-files.

  23. MarcoFalke merged this on May 3, 2021
  24. MarcoFalke closed this on May 3, 2021
  25. jonatack deleted the branch on May 3, 2021
  26. PastaPastaPasta referenced this in commit 5c0dc92760 on Jun 27, 2021
  27. PastaPastaPasta referenced this in commit 3927795629 on Jun 28, 2021
  28. PastaPastaPasta referenced this in commit 9580f65804 on Jun 29, 2021
  29. PastaPastaPasta referenced this in commit 2efbfb6224 on Jul 1, 2021
  30. PastaPastaPasta referenced this in commit bcc76f109c on Jul 1, 2021
  31. gwillen referenced this in commit bfdf82b18f on Jun 1, 2022
  32. bitcoin locked this on Aug 18, 2022
Contributors
jonatacklaanwjSjorsMarcoFalkepracticalswifthebastoadamjonas
Labels
Docs
Linked (view graph)
#21842 Update reviewers file: Remove too broad (`*.sh`) entry. Keep narrower entries.
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-19 06:53 UTC