skunk icon indicating copy to clipboard operation
skunk copied to clipboard

Add option to turn off emojis in exceptions

Open kubukoz opened this issue 1 year ago • 2 comments

The flood of 🔥 emojis looks cool and definitely draws attention when you hit an issue in local development, but it's not well supported in all CI systems, and certainly not in all logging systems. There should be a way to tweak this behavior, perhaps even with sane defaults such as disabling the fires when CI=1 or based on some terminal variables.

kubukoz avatar Oct 23 '24 14:10 kubukoz

In addition, although emojis work in apps like DataDog, the ANSI escape codes don't - these should probably be conditionally stripped as well.

Image

kubukoz avatar Mar 12 '25 17:03 kubukoz

Some things to consider on this one:

  • should this be configured globally (e.g. via environment variable or system property) or via session level configuration parameter (e.g. Session.Builder[F].withPrettyInfo(false)...)
  • emojis and ANSI escapes are used pervasively throughout the code base in both exceptions and debug statements - should the enablement/disablement be communicated to all places strings are constructed? Or should strings get constructed as-is and then stripped/scrubbed before printing?

mpilquist avatar Mar 18 '25 14:03 mpilquist