plots modify: change --no-csv-header to --no-header?

[jorgeorpinel]

Extracted from treeverse/dvc.org#1382 (review)

should it be just --no-header? - @shcheklein

or --no-table-header since TSV is also supported. This way it remains obvious that it only applies to tabular metric files. But I could also support --no-header — cleaner.

• me

I like the --no-table-header - @pared

Please provide information about your setup

DVC 1.0

[efiop]

Guys, so maybe --no-header after all? Or do you see a different usecase for --no-header that will force us to create --no-table-header in the future to differentiate the use cases? @pared

[shcheklein]

+1 for --no-header, at least --no-csv-header feels wrong since it's not CSV only

[sahilbhosale63]

Hi, Can I work on this issue?

[pared]

I don't see a future use case for --no-header as of today, let's do it. I guess if there would be more important and viable use case for that option, we would know by now.

[efiop]

Hi @sahilbhosale63 !

Sorry, don't mean to discourage, but this one is already WIP by me, so, if you still would like to contribute, please take a look at good-first-issue and help-needed labels in our issues list.

Thank you! 🙏