search

muesli / termenv

Advanced ANSI style & color support for your terminal applications
MIT License
1.76k stars 73 forks source link

fix,feat: assume tty using environment variables #133

Open aymanbagabas opened 1 year ago

aymanbagabas commented 1 year ago

Make Termenv aware of TERMENV_TTY and assume tty using environment variables.

This also checks the actual value of CI instead of its length.

Fixes: 1111971edffa ("Don't run OSC queries on CI")

muesli commented 1 year ago

That would however reintroduce issues with CIs like CircleCI.

aymanbagabas commented 1 year ago

That would however reintroduce issues with CIs like CircleCI.

Changed the PR to add a new TERMENV_TTY variable and check the value of CI