On 20.06.25 16:41, David G. Johnston wrote:
I sense there could be some confusion whether such draft patches
should go into the regular commit fest or the draft commit fest, and
then also when they should move between them.
Draft CF patches with “Needs Review” are looking for feedback from
others or otherwise aid in development for a patch that isn’t ready to
be committed even if said review turns up nothing or is otherwise fully
resolved. Patches in Drafts are never marked Ready to Commit, they get
moved to Open first.
It will be nice if people spend time providing reviews/feedback to
patches in Drafts when requested.
It’s purely the author’s judgement on the readiness of the patch,
whether absent our policy they would mark it ready to commit or not. If
they believe it is it should be moved to open, if no, it should remain
in drafts. This is mostly like what happens today but with a clear
delineation between reviews to help and reviews to approve commit-ability.
Otherwise, it’s a place where author patches can sit without having to
be bumped to the next cf every other month and where an author patch can
be ignored by everyone else not authoring it.
I don't know about this. This could become an ongoing source of
confusion, without any clear benefit. Either the draft and the "real"
commitfest are going to be indistinguishable, because they are just two
places to look for stuff to review in various phases of maturity. Or
the draft commitfest is just not going to get any attention, which will
be annoying for those who put things there hoping to get attention.
