Page MenuHomePhabricator

Make objects implementing BuildableInterface produce a BuildableEngine
ClosedPublic

Authored by epriestley on Apr 3 2018, 11:46 AM.
Tags
None
Referenced Files
F14080177: D19278.diff
Fri, Nov 22, 11:53 AM
Unknown Object (File)
Tue, Nov 19, 5:12 AM
Unknown Object (File)
Sat, Nov 16, 7:11 PM
Unknown Object (File)
Sat, Nov 16, 7:11 PM
Unknown Object (File)
Sat, Nov 16, 7:11 PM
Unknown Object (File)
Sat, Nov 16, 6:46 PM
Unknown Object (File)
Oct 11 2024, 11:10 AM
Unknown Object (File)
Oct 9 2024, 9:16 AM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13110. Currently, build status is published the same way for every Buildable by the BuildEngine.

I want to change this to delegate publishing to each Buildable, particularly so that Differential may use more detailed rules for handling builds and drafts.

Rather than add additional methods to the existing BuildableInterface, add an engine generator method instead. This is a pattern which has seen more use recently (e.g., in Ferret) and lets us pay a little more upfront to pull complex pieces of logic out of the main class and let them use inheritence more easily. If we had Traits that might cover this to some degree.

I'd expect to eventually reduce the size of BuildableInterface and move the CircleCI and BuildKite interfaces so that the BuildableEngine implements them instead of the main object.

Here, this new engine does nothing and is never instantiated. In upcoming changes, publishing logic will move into it so that Differential can handle publishing differently.

Test Plan

Ran arc liberate, loaded pages, grepped for BuildableInterface.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable