Page MenuHomePhabricator

Use tabs on build targets and allow build steps to have a description
ClosedPublic

Authored by hach-que on Jul 31 2014, 5:35 AM.
Tags
None
Referenced Files
F13143557: D10093.diff
Fri, May 3, 7:34 AM
Unknown Object (File)
Wed, Apr 24, 4:19 AM
Unknown Object (File)
Mon, Apr 22, 8:55 PM
Unknown Object (File)
Fri, Apr 19, 2:05 AM
Unknown Object (File)
Fri, Apr 19, 2:05 AM
Unknown Object (File)
Fri, Apr 19, 2:05 AM
Unknown Object (File)
Fri, Apr 19, 2:05 AM
Unknown Object (File)
Fri, Apr 12, 8:59 PM
Subscribers

Details

Summary

Ref T1049. This uses tabs on build targets to hide the configuration details and variables by default, instead promoting the target name, it's status and a description of the build step. The description is a new field on each build step.

The primary advantage of having a description on build steps is that DevOps can configure appropriate description information (including any troubleshooting information for build failures) on build steps, and developers who have builds fail against their code review can then look at this information.

Test Plan

Viewed a build plan and saw the appropriate information.

Diff Detail

Repository
rP Phabricator
Branch
target-tabs
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1959
Build 1960: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que retitled this revision from to Use tabs on build targets and allow build steps to have a description.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Make sure basic information is always shown

epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 31 2014, 7:41 PM
hach-que updated this revision to Diff 24294.

Closed by commit rPdfa9b27a949c (authored by @hach-que).