feat(factory): turn review comments on the delivery PR into follow-up work #759

Open
opened 2026-09-29 09:27:02 +02:00 by dries · 0 comments
Owner

Follow-up to #757. Unresolved review comments on a Factory delivery PR should create fix Issues on the same Factory branch, then refresh the delivery. This is the open question in the old factory-feedback-handling research: who decides a comment is required, where optional feedback waits, and how to react to a merge request mid-run.

Scope:

  • poll delivery PR review comments (forge clients already poll merge gates)
  • policy for required vs optional comments (e.g. only 'request changes' reviews, or a label)
  • create fix Issues, re-run verification and delivery on the same PR
  • avoid loops on bot comments

Useful 4/5 · Effort M–L

Follow-up to #757. Unresolved review comments on a Factory delivery PR should create fix Issues on the same Factory branch, then refresh the delivery. This is the open question in the old factory-feedback-handling research: who decides a comment is required, where optional feedback waits, and how to react to a merge request mid-run. Scope: - poll delivery PR review comments (forge clients already poll merge gates) - policy for required vs optional comments (e.g. only 'request changes' reviews, or a label) - create fix Issues, re-run verification and delivery on the same PR - avoid loops on bot comments Useful 4/5 · Effort M–L
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dries/ocman#759
No description provided.