Page MenuHomePhabricator

In Differential, render status for any active Drydock repository operation, not just "Land" operations
ClosedPublic

Authored by epriestley on Jul 25 2017, 1:56 PM.
Tags
None
Referenced Files
F18854686: D18276.id.diff
Nov 1 2025, 12:26 AM
F18773740: D18276.id.diff
Oct 9 2025, 12:19 PM
F18771018: D18276.diff
Oct 8 2025, 4:52 PM
F18733689: D18276.diff
Sep 30 2025, 10:40 PM
F18617611: D18276.id43955.diff
Sep 14 2025, 9:40 PM
F18598683: D18276.diff
Sep 13 2025, 5:35 AM
F18348198: D18276.diff
Aug 26 2025, 4:26 PM
F18115152: D18276.diff
Aug 13 2025, 6:26 AM
Subscribers
None

Details

Summary

See PHI18. Third parties can currently define other types of Drydock operations (like "Merge Check" or "Cherry-Pick") but we won't show them in the UI.

This is a simple change which improves third-party support for now. These kinds of operations generally make sense in the upstream, but the pathways to support are longer.

Test Plan
  • Verified that there are no other types of repository operation which we'd want to exclude in the upstream today by reviewing the "Repository Operation" subclasses.
  • Will click some buttons in production to make sure this works.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable