Commits on Aug 26, 2023

1. fix: read fore/background colors on first access …

   Caching the colors in OutputOption makes the order of options important.
   For example, if you're using WithUnsafe _after_ WithColorCache, the
   colors wouldn't respect the `unsafe` option since the colors get cached
   _during_ the execution of WithColorCache option.
   
   Another example, is the use of a custom environ using the WithEnvironment
   option. If WithEnvironment comes _after_ WithColorCache, reading the
   terminal colors won't respect the values in the environment.
   
   Instead, don't read the colors in WithColorCache and make the user
   opt-in if they want to cache the colors right after creating a new
   output using ForegroundColor and BackgroundColor.
   
   ```go
   // To cache the colors after creating an output
   output := NewOutput(os.Stderr)
   _ = output.ForegroundColor()
   _ = output.BackgroundColor()
   ```

   

   aymanbagabas committed Aug 26, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for fbe6f55
   • Browse repository at this point

   Copy the full SHA

   fbe6f55 View commit details

   Browse the repository at this point in the history