fix(output): check for nil tty writer #97

aymanbagabas · 2022-10-25T14:17:22Z

if tty is nil any subsequent call to Write() will cause a panic
Also introduce OutputOption type instead of func(*Output)

if tty is nil any subsequent call to `Write()` will cause a panic

aymanbagabas requested a review from muesli October 25, 2022 14:17

aymanbagabas force-pushed the check-nil branch 2 times, most recently from 4c655bf to 0e2a510 Compare January 31, 2023 18:11

aymanbagabas added 2 commits January 31, 2023 13:13

fix(output): check for nil tty writer

d902137

if tty is nil any subsequent call to `Write()` will cause a panic

feat: use OutputOption

08c00a5

aymanbagabas force-pushed the check-nil branch from 0e2a510 to 08c00a5 Compare January 31, 2023 18:13

aymanbagabas merged commit 925ea91 into master Jan 31, 2023

Provide feedback