doc: empty REVIEWERS file #25613

1. 
   fanquake commented at 12:28 pm on July 14, 2022: member

   It seems that it’s time for our experiment with this file to come to an end.

   See discussion here: https://github.com/bitcoin/bitcoin/pull/25560/files#r915491743.

2. fanquake added the label Docs on Jul 14, 2022
3. kristapsk approved
4. 
   kristapsk commented at 12:33 pm on July 14, 2022: contributor

   ACK cdbc68c1cd9b8228f8cdee9f61c20639325fb67d. It’s kinda CODEOWNERS, don’t think Bitcoin Core should have such thing.
5. 
   Sjors commented at 1:50 pm on July 14, 2022: member

   No strong preference. I usually look at git blame if I feel the need to nag more reviewers.
6. 
   MarcoFalke commented at 2:01 pm on July 14, 2022: member

   Yes, git blame is best if you want to manually find reviewers.

   However, this list was made for people to opt-in to bot notifications (as it is explained in the first line of the file), but then some people on the list complained about being notified by a bot. :man_shrugging:

   If there is anyone in this list who prefers those bot notifications, I think we should keep them in this list and remove everyone else.

7. 
   MarcoFalke commented at 2:31 pm on July 14, 2022: member

   Currently, the format of the comment looks like this: #25614 (comment)

   This may be interesting to people that are not subscribed to the repo, but want to subscribe to changes to a specific file or folder.

   My suggestion would be to keep the file and remove all contents for now, then leave the pull open for two weeks before merge. In those two weeks, people can explicitly opt-in by leaving a comment to be (re-)added to the file with the understanding that they will receive some kind of bot notifications. Also, anyone can open a pull request in the future to opt-in.

8. 
   DrahtBot commented at 7:02 pm on July 14, 2022: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   No conflicts as of last run.

9. 
   fanquake commented at 7:56 am on July 30, 2022: member

   My suggestion would be to keep the file and remove all contents for now, then leave the pull open for two weeks before merge. In those two weeks, people can explicitly opt-in by leaving a comment to be (re-)added to the file with the understanding that they will receive some kind of bot notifications.

   Should we wait any longer?

10. 
    MarcoFalke commented at 8:02 am on July 30, 2022: member

    I’d also say to keep the empty file in the repo for another two months to give people a chance to add themselves (opt-in to notifications) if they want.
11. doc: empty REVIEWERS file

    It seems that it's time for our experiment with this file to come to an
    end.
    
    See discussion here:
    https://github.com/bitcoin/bitcoin/pull/25560/files#r915491743.

    4d06fc4bed
12. fanquake force-pushed on Jul 30, 2022
13. 
    fanquake commented at 8:06 am on July 30, 2022: member

    I’d also say to keep the empty file in the repo

    Pushed to just empty the file.

14. 
    MarcoFalke commented at 8:07 am on July 30, 2022: member

    ACK 4d06fc4bed12ca119208ee53369719f52b0f7098 seems best to have devs opt-in to this explicitly by themselves
15. MarcoFalke renamed this:
    doc: remove REVIEWERS
    doc: empty REVIEWERS file
    on Aug 1, 2022
16. MarcoFalke merged this on Aug 1, 2022
17. MarcoFalke closed this on Aug 1, 2022

    ---

18. fanquake deleted the branch on Aug 1, 2022
19. sidhujag referenced this in commit 52f531f3eb on Aug 1, 2022
20. MarcoFalke locked this on Feb 10, 2023

Contributors
fanquake kristapsk Sjors MarcoFalke DrahtBot

Labels
Docs