1610403 - Allow to submit a part of a stack as "wip"

Status: RESOLVED FIXED

(Conduit :: moz-phab, enhancement, P3)

Description

[Mitchell Hentges [:mhentges] 🦀]

Disclaimer: I might be using moz-phab incorrectly.

To reproduce:

1. Make a solid change, submit it with moz-phab
2. The revision shows up in Phabricator. Assign reviewers, feel happy that you made a good patch
3. Some time later, work on a new patch. This patch depends on the previous, stable patch that you just submitted above^. You want some feedback from your team on this patch, so you submit it to Phabricator with moz-phab --wip and link it to your teammates.
4. The original patch at the bottom of the stack is marked as "changes planned", even though only the last patch is "wip".

Expected behaviour

• The patch that I submitted with --wip should be marked as "changes planned".
• The patches below this patch shouldn't be marked as "changes planned".

Comment 1

[Piotr Zalewa [:zalun]]

Switches in moz-phab are modifying the entire stack.
We could be detecting the WIP state from the commit title - "WIP " or `"WIP:".

Comment 2

[Zeid Zabaneh [:zeid]]

Might be good to look into standardizing the WIP flag -- e.g. if it commit message starts with WIP: as suggested, mark it is draft, or if either --wip is passed or a draft revision is detected during submit make sure to prepend the first line starts with WIP: if it's not already there.

Comment 3

[:glob ✱]

(In reply to Zeid Zabaneh [:zeid] from comment #2)

Might be good to look into standardizing the WIP flag -- e.g. if it commit message starts with WIP: as suggested, mark it is draft, or if either --wip is passed or a draft revision is detected during submit make sure to prepend the first line starts with WIP: if it's not already there.

I like this idea, however updating the local commit description for any commit other than HEAD is an expensive operation that we should avoid if possible.

Prepending the commit description sent to Phabricator should be trivial.

Comment 5

[:glob ✱]

Attachment: Bug 1610403 - Shift WIP tracking from stack to commit level; r=zeid!

• adds wip attribute to each commit in a stack
• set wip if any of the following is true:
  • --wip command line argument provided
  • commit description starts with WIP:, WIP, or is just WIP
  • commit is lacking reviewers
• prefix revision summary with WIP: for all WIP submissions
• handle revision state transisions into and out of WIP
• some minor UI changes

Comment 6

[:glob ✱]

https://github.com/mozilla-conduit/review/commit/e927434d576a0b872fdf0cc966b55d5fac4d9f7b