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)
Thu, Dec 19, 5:22 PM
Unknown Object (File)
Tue, Dec 17, 4:10 AM
Unknown Object (File)
Dec 13 2024, 7:46 PM
Unknown Object (File)
Dec 12 2024, 12:30 AM
Unknown Object (File)
Dec 7 2024, 10:20 PM
Unknown Object (File)
Dec 4 2024, 11:30 PM
Unknown Object (File)
Dec 4 2024, 8:49 AM
Unknown Object (File)
Nov 29 2024, 8:34 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