Ben Shaver

Results 17 comments of Ben Shaver

> @bpshaver please run `make fmt` on your changes so that `check-fmt` check can succeed in CI. Done. Does this seem like a sensible change? If so, I'll look through...

@pavdmyt I do not understand how this is more convenient. The idea of detecting whether the target stream is a TTY and changing behavior or issuing a warning is a...

I understand now, thanks! I would call the `.text`, etc. _accompanying text_. Under normal circumstances, it is right next to the spinner. You're assuming that when the output is redirected...

_accompanying text_ is my name for whatever text normally goes right next to the spinner. You called it _useful text_, which is begging the question with respect to where it...

Seems like @pavdmyt disagreed with my reasoning and the discussion ended.

Agreed. The first thing I noticed when I looked at the code was the lack of type hints and the violation of standard PEP 8 formatting guidelines, such as a...

@wgong the idea that you would use an LLM to do formatting when Black and Ruff exist for that purpose is strange to me. It tickles me to imagine someone...