Skip to content
This repository has been archived by the owner on Oct 25, 2021. It is now read-only.

[cli] Provide options to disable columns #213

Closed
matthiasbeyer opened this issue Jan 6, 2018 · 6 comments
Closed

[cli] Provide options to disable columns #213

matthiasbeyer opened this issue Jan 6, 2018 · 6 comments
Labels

Comments

@matthiasbeyer
Copy link
Contributor

The CLI currently provides flags for adding more output columns (-DPOLT) though no option for removing columns from the output.

Usecase: I have TST entries with many parts. The output wraps several times (on a 1080p monitor with normal text size I get lines that wrap 6 times because of >30 parts).

@vitiral
Copy link
Owner

vitiral commented Jan 6, 2018

I'm unsure what you would like.

The "default" view is name+parts

# art ls                                                                                                                                                                              
|  | DONE TEST | NAME                    | PARTS                                                                                                                                                                  
|--|  83%  71% | REQ-data                | REQ-data-family, REQ-data-type, SPC-data

If you want just one of them, you specify it:

# art ls -L                                                                                                                                                                           
|  | DONE TEST | NAME                    | IMPLEMENTED  

When you pass the -A flag, additional flags disable columns. Interestingly, if you try to disable all of them it still displays parts, which is definitely a bug:

art ls -ADPOLT
|  | DONE TEST | NAME                    | PARTS   
|--|  83%  71% | REQ-data                | REQ-data-family, REQ-data-type, SPC-data

Do you mean you would like to ONLY display the name? I don't think that is currently supported (because of the above bug). I could add a
-N flag, but then I would have to support that removing the name when used with -A, which I suppose is fine (maybe someone might want to do that?).

@vitiral vitiral added the bug label Jan 6, 2018
@matthiasbeyer
Copy link
Contributor Author

I missed the "deactivate" in the help line... I should read more carefully.

    -A, --all       "all" fields. If set, additional field flags will *deactivate* fields

Everything is right. Do you want to close this or leave it open for the found bug?

@vitiral vitiral closed this as completed Nov 6, 2018
@vitiral
Copy link
Owner

vitiral commented Nov 6, 2018

I think the parts bug is fixed

@vitiral
Copy link
Owner

vitiral commented Nov 6, 2018

Actually it's worse now... deactivating doesn't work at all... ☹️

@vitiral vitiral reopened this Nov 6, 2018
@vitiral
Copy link
Owner

vitiral commented Nov 6, 2018

oh wait, nevermind. Deactivating only doesn't work for subnames+impl (new fields!)

@vitiral
Copy link
Owner

vitiral commented Nov 6, 2018

and... nevermind again. It's just that subnames+impl are not documented in the --fields option so I missed them. This works:

$ art ls -A -NFPOTIS
spc%  tst%
60.0  34.7
45.7  33.7

I opened #240 for the doc bug

@vitiral vitiral closed this as completed Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants