table icon indicating copy to clipboard operation
table copied to clipboard
Published 2 months ago • verified publisher icon rodaine

Allow for optional headers

Open punnie opened this issue 2 years ago • 1 comments

This PR adds the flag PrintHeaders to the table struct that specifies whether headers should be printed or not.

This may be useful when output is being piped to other processes, where having a header is less desirable, and saves from doing a call to head (or tail) before processing.

Thank you for this wonderful library. Please let me know what you think about this, and if any more changes are necessary.

punnie avatar Aug 20 '23 21:08 punnie

@rodaine would it be possible to get this PR caught up and merged? I am happy to take it on if @punnie is unable. This functionality would be beneficial in a project of mine.

IdlePhysicist avatar Apr 24 '24 03:04 IdlePhysicist

Hey y'all!

Hadn't realised there was a call to action here, sorry for the delay.

Resolved the conflict by interacting with a new feature added in PR #40 in a way that made sense to me: do not print the header separator if headers are not to be printed. Please let me know if this is the intended behaviour or if there are other changes I need to do.

Thanks!

punnie avatar Aug 02 '24 13:08 punnie

Thanks for the patch (and everyone's patience on getting this merged)!

rodaine avatar Aug 02 '24 14:08 rodaine