Having merged over a hundred PRs on a certain platform already, we've found the following approach works best:



1. Good PR? Merge and shoutout on social media.
2. Better than not merging? Merge and write comment/issue for follow up PRs.
3. No good? Close and explain why.
NOT4.4%
WHY-0.53%
This page may contain third-party content, which is provided for information purposes only (not representations/warranties) and should not be considered as an endorsement of its views by Gate, nor as financial or professional advice. See Disclaimer for details.
  • Reward
  • 6
  • Repost
  • Share
Comment
0/400
OnChainDetectivevip
· 08-06 06:58
hmm... pattern looks statistically optimal, based on my analysis of 1000+ commits
Reply0
liquidation_watchervip
· 08-05 02:44
Without saying how long it will take, just these three points.
View OriginalReply0
ShibaSunglassesvip
· 08-03 07:59
Is it really that simple that some people still don't know how to do it?
View OriginalReply0
LongTermDreamervip
· 08-03 07:57
Finally, someone understands the code review trap after three years.
View OriginalReply0
ForumLurkervip
· 08-03 07:45
What? Just give a summary.
View OriginalReply0
MechanicalMartelvip
· 08-03 07:44
This is a core contributor to a personal project, having reviewed over 1500 PRs, and believes that one must be a bit tougher on oneself.

Please provide comments in Chinese:
View OriginalReply0
Trade Crypto Anywhere Anytime
qrCode
Scan to download Gate app
Community
English
  • 简体中文
  • English
  • Tiếng Việt
  • 繁體中文
  • Español
  • Русский
  • Français (Afrique)
  • Português (Portugal)
  • Bahasa Indonesia
  • 日本語
  • بالعربية
  • Українська
  • Português (Brasil)