Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --no-single-file-header flag #1395

Closed

Conversation

niklasmohrin
Copy link
Contributor

Closes #1311

Hey there! This was a pretty straight-forward change, let me know if I missed anything or if you want to have this handled differently :)

@sharkdp
Copy link
Owner

sharkdp commented Dec 21, 2020

Thank you very much for your contribution!

@eth-p @keith-hall I'm not really convinced we should a new command-line option for this. Having a header for a single file is not really useful if someone is really just calling bat example.rs. But it's useful in a lot of other scenarios, e.g. when using bat to copy&paste code (or a screenshot of it) somewhere. So I don't want to disable the single file header by default.

I wonder if we could extend the --style option (again) to support this somehow?

@sharkdp
Copy link
Owner

sharkdp commented Jan 2, 2021

I'd rather not implement this for now. Thank you, @niklasmohrin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show headers only when multiple files are passed
2 participants