Confirmed users
1,927
edits
("Done" is now checked by default when the author leaves a comment, so removing this entry) |
(Add note about needing to rerun "arc diff" after a backout) |
||
Line 40: | Line 40: | ||
[[File:Closed revision actions.png|320px|Closed revision actions]] | [[File:Closed revision actions.png|320px|Closed revision actions]] | ||
Note that you will need to run "arc diff" again to update the patch even if you don't need to change it (e.g. the bustage was in an earlier commit). This is because Phabricator updates the revision after it lands but does not provide the necessary metadata that Lando needs. See {{bug|1489126}}. | |||
See also https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#other-revision-actions for descriptions of other non-review-related actions you might want to take on a revision. | See also https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html#other-revision-actions for descriptions of other non-review-related actions you might want to take on a revision. |