Skip to content

Conversation

@magnus-ISU
Copy link
Collaborator

I am not going to merge this yet because it makes a potentially very undesirable change: It makes lolcat automatically wrap words. Words are defined as space-separated strings. So

$ echo words that are awesome and very cool and all on one line including one super long word: aoeushaeoshusaohueoahush > test_file
$ lolcat --terminal-size 10 test_file

prints

words 
that are 
awesome 
and very 
cool and 
all on 
one line 
including
one super
long 
word: 
aoeushaeo
shusaohue
oahush

lolcat is about printing (fun) things for humans. So I think reflowing output is not necessarily a bad thing. But it may be very different from what is expected for a drop-in replacement for the original lolcat. Therefore it might be wiser to put it behind a -w flag to turn it on.

However, I think also that making useful upgrades require random command-line flags to turn on is silly and a big part of why people love ripgrep and bat and exa etc.

I do think that the only output which this particularly breaks is probably already broken though. So I am in favor of it myself, just don't want to make such a change randomly.

Therefore, I am conflicted. I would like feedback on whether this should be merged or not. @ur0

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants