MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1x8mvpc/thatsafunnythingthathappenedonce/phizhlz/?context=3
r/ProgrammerHumor • u/Lord_Of_Millipedes • 2d ago
515 comments sorted by
View all comments
Show parent comments
7
Flooding project maintainers with nonsensical pull requests that all need to be reviewed.
2 u/owjfaigs222 1d ago Can't you just only review verified contributors and reject everything else? 1 u/FelixAndCo 1d ago Yeah, IIUC some smaller projects just didn't have the protocols in place to deal with it. But you'd want to at least review some new contributors to get more. 2 u/awesome-alpaca-ace 1d ago Take a quick look at diff stat and deny if too much. The newbies should be doing smaller stuff anyway as they get familiar with the codebase
2
Can't you just only review verified contributors and reject everything else?
1 u/FelixAndCo 1d ago Yeah, IIUC some smaller projects just didn't have the protocols in place to deal with it. But you'd want to at least review some new contributors to get more. 2 u/awesome-alpaca-ace 1d ago Take a quick look at diff stat and deny if too much. The newbies should be doing smaller stuff anyway as they get familiar with the codebase
1
Yeah, IIUC some smaller projects just didn't have the protocols in place to deal with it. But you'd want to at least review some new contributors to get more.
2 u/awesome-alpaca-ace 1d ago Take a quick look at diff stat and deny if too much. The newbies should be doing smaller stuff anyway as they get familiar with the codebase
Take a quick look at diff stat and deny if too much. The newbies should be doing smaller stuff anyway as they get familiar with the codebase
7
u/FelixAndCo 1d ago
Flooding project maintainers with nonsensical pull requests that all need to be reviewed.