Page MenuHomePhabricator

Move "Wait for Previous Commits to Build" out of prototype
ClosedPublic

Authored by epriestley on Jul 30 2020, 7:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 3:29 PM
Unknown Object (File)
Thu, Apr 25, 2:11 AM
Unknown Object (File)
Sun, Apr 21, 10:50 PM
Unknown Object (File)
Sat, Apr 20, 2:13 PM
Unknown Object (File)
Thu, Apr 18, 11:13 PM
Unknown Object (File)
Thu, Apr 18, 8:43 AM
Unknown Object (File)
Mar 28 2024, 7:53 PM
Unknown Object (File)
Mar 27 2024, 12:40 PM
Subscribers
Restricted Owners Package

Details

Summary

Although I'm not entirely thrilled about doing flow control like this (as an actual action in a build plan), I believe this build step works correctly and there's no fancy replacement mechanism on the immediate horizon, and this didn't send us down a slippery slope of Turing-complete builds encoded without real structure or context. Just kick it out of prototype.

(Other approaches which might be better in the long run are things like "this is a top-level behavior on the build plan itself" and/or "build plans are written in a DSL, not a Javascript UI".)

Test Plan

Added a new build step, saw this as an option in the "Flow Control" section.

Diff Detail

Repository
rP Phabricator
Branch
wait1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24814
Build 34235: Run Core Tests
Build 34234: arc lint + arc unit