Page MenuHomePhabricator

Added `build.id` variable and other improvements / fixes
ClosedPublic

Authored by hach-que on Nov 9 2013, 2:59 AM.
Tags
None
Referenced Files
F14064662: D7540.diff
Tue, Nov 19, 12:41 AM
F14051785: D7540.diff
Fri, Nov 15, 3:55 AM
F14040708: D7540.diff
Mon, Nov 11, 12:35 PM
F14024420: D7540.diff
Thu, Nov 7, 9:20 AM
F14006934: D7540.diff
Mon, Oct 28, 9:32 PM
F14005830: D7540.diff
Sun, Oct 27, 8:26 PM
F13985870: D7540.id17017.diff
Oct 20 2024, 11:19 PM
F13966297: D7540.id17016.diff
Oct 16 2024, 7:26 AM

Details

Summary

This adds a build.id variable, cleans up the naming convention of other variables and also fixes an issue in the remote command to read the buffers after the command finishes.

Test Plan

Ran a build with /bin/echo ${build.id} and saw the build ID come through.

Diff Detail

Branch
buildid-variable
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php:233PHL1Unknown Symbol
Unit
No Test Coverage

Event Timeline

epriestley closed this revision.

Closed by commit rP3eaac9eda9d5 (authored by @hach-que, committed by @epriestley).