Hacker News Reader

I see PRs with titles prefixed "DO NOT MERGE" or "WIP". People want the UI that goes along with a PR, prior to it being ready to merge.

View on HN

Right, this is a much better approach than such titles (or equivalent labels, which I've also seen), because it directly impacts GitHub's "ready to merge" checks at the bottom of the PR (without needing an outside bot/service for it).

You don't have a review mandate before merge?

Yeah, but people might approve it for merge when it is not obvious that it isn't ready.

Wouldn't this same person do the same for a Draft PR? i.e. if they're just reviewing things wouldn't they just set the Draft PR 'ready to review' and then review it?

Or maybe I'm confused by your answer, are people commonly reviewing PR's without being asked?

What the OP is saying is that someone can review a PR even when it's not ready by accident (not seeing WIP in title/tag), while with a draft, the reviews are disabled until its ready for review.

This basically is a better WIP system than before because it can prevent accidental reviews and wasting time of re-reviewing when it actually is ready.

Reply on news.ycombinator.com