Page MenuHomePhabricator

Give data exporters a header row
ClosedPublic

Authored by epriestley on Jan 29 2018, 2:35 AM.
Tags
None
Referenced Files
F18225236: D18952.diff
Wed, Aug 20, 1:49 AM
F18109511: D18952.id45451.diff
Mon, Aug 11, 2:13 PM
F18103075: D18952.id45476.diff
Sun, Aug 10, 1:32 AM
F18097975: D18952.id.diff
Fri, Aug 8, 5:17 PM
F18093384: D18952.diff
Thu, Aug 7, 5:56 PM
F17953160: D18952.id45476.diff
Fri, Aug 1, 2:49 AM
F17942765: D18952.diff
Thu, Jul 31, 7:08 AM
F17941095: D18952.id45451.diff
Thu, Jul 31, 5:02 AM
Subscribers
None

Details

Summary

Depends on D18951. Ref T13049. When we export to CSV or plain text, add a header row in the first line of the file to explain what each column means. This often isn't obvious with PHIDs, etc.

JSON has keys and is essentially self-labeling, so don't do anything special.

Test Plan

Exported CSV and text, saw new headers. Exported JSON, no changes.

Diff Detail

Repository
rP Phabricator
Branch
export2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19235
Build 25986: Run Core Tests
Build 25985: arc lint + arc unit