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
Unknown Object (File)
Mon, Jun 9, 12:51 PM
Unknown Object (File)
Fri, Jun 6, 12:56 PM
Unknown Object (File)
Mon, May 19, 12:13 AM
Unknown Object (File)
Apr 30 2025, 2:02 PM
Unknown Object (File)
Apr 30 2025, 7:24 AM
Unknown Object (File)
Apr 29 2025, 12:00 PM
Unknown Object (File)
Apr 28 2025, 4:20 PM
Unknown Object (File)
Apr 25 2025, 5:18 PM
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