HomePhorge

Reduce laziness for "Mark Committed"

Description

Reduce laziness for "Mark Committed"

Summary:

  • Enforce proper workflow rules.
  • Fix a derp-bug with patches.

Test Plan:

  • Tried to mark a revision I didn't own.
  • Tried to mark a revision already marked committed.
  • Tried to mark a revision otherwise not accepted.
  • Verified daemon can override workflow rules and mark from arbitrary states.

Reviewers: btrahan, Makinde

Reviewed By: Makinde

CC: aran, epriestley

Maniphest Tasks: T948

Differential Revision: https://secure.phabricator.com/D1809

Details

Provenance
epriestleyAuthored on Mar 7 2012, 10:20 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP62f18914db11: Add "mark committed" action to the web UI
Branches
Unknown
Tags
Unknown

Event Timeline