Constructing tables manually just isn't fun.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- Restricted Diffusion Commit
rPb7b045830307: Convert `./bin/storage status` to use `PhutilConsoleTable`.
./bin/storage status phabricator:db.audit Applied db audit phabricator:db.calendar Applied db calendar phabricator:db.chatlog Applied db chatlog phabricator:db.conduit Applied db conduit phabricator:db.countdown Applied db countdown phabricator:db.daemon Applied db daemon phabricator:db.differential Applied db differential phabricator:db.draft Applied db draft phabricator:db.drydock Applied db drydock phabricator:db.feed Applied db feed phabricator:db.file Applied db file phabricator:db.flag Applied db flag phabricator:db.harbormaster Applied db harbormaster ...
This probably isn't ready to land yet, we should fix PhutilConsoleTable to truncate columns which would otherwise cause overflow.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- storagestatustable
- Lint
Lint Passed - Unit
Tests Skipped - Build Status
Buildable 1184 Build 1184: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
Comment Actions
(We could likely also convert ./bin/storage probe, but the output format is not a conventional table)
Comment Actions
This probably isn't ready to land yet, we should fix PhutilConsoleTable to truncate columns which would otherwise cause overflow.
I think this is fine -- it's sort of bad to truncate the paths, and the non-truncation behavior is no worse than what we do today.