logoalt Hacker News

gus_massayesterday at 10:47 PM1 replyview on HN

I'd just limit the total. Otherwise people will use it to game the system.


Replies

moritzwarhiertoday at 12:31 AM

Or using it correctly... depends on what your system is, or more accurately, your policy on reviewing or quickly merging PRs with "Draft" status.

As long as it's taken as an indicator for WIP, it works. It just doesn't work when acting illiterate of this distinction; and I have often have had PRs switched to "ready", reviewed + merged in a couple of hours.

But when the change list grows, and the PR ages, while still being intentionally maintained, the Draft signal is strong and helpful IMO. Switching an old Draft PR to "ready" after reviving it with changes seems like a useful signal to me.

show 1 reply