vito / progrock

progress ui that rocks
Apache License 2.0
62 stars 7 forks source link

fix: ensure that NewOutput always assumes a TTY #14

Closed jedevc closed 7 months ago

jedevc commented 7 months ago

See https://github.com/dagger/dagger/pull/7070. This replaces https://github.com/dagger/dagger/issues/7047.

The description of this function currently lies. Without this, the isTTY function in termenv will return false, so we never get colors.