I noticed that one of the PRs assigned to me (https://github.com/rust-lang/rust/pull/75645) was converted to draft by triage team.
Why was it done?
Draft status is redundant when we already have waiting-on labels, and is not useful neither to reviewer nor to author.
It also confuses automation (or at least it did in the past).
Large portion of PRs actually arrive in draft status and need significant rework, without being marked as draft.
This is a usual matter of communication between the author and reviewer, and switching between waiting-on-author and waiting-on-review is enough to facilitate it.
@Vadim Petrochenkov Not related to your specific PR, but it’s worth noting that GitHub is planning on making the draft state of PRs more meaningful with pull request revisions. https://github.com/github/roadmap/issues/54
not sure why this one was marked as draft (it doesn't have to) but normally it's done for experiments or to signify that it is work in progress (waiting-on-author would suffice but it's also used for other things such as when there are changes requested or conflicts, so draft is a more specific case)
I did the conversion, because it seems the PR is not quite ready for review(needs investigation and maybe large portions of changes)? Since the PR authored by rust-lang member, we cannot ping that member, instead we'll need to bump the labels again and again if it's in S-waiting-for-author state but not as a draft PR (if it's in draft PR state we as triagers can just skip it).
Last updated: Jan 26 2022 at 14:20 UTC