Closed kozross closed 3 months ago
@SeungheonOh - I don't think so either, but @peter-mlabs is the best judge of this.
I think configurable log levels would be a nice-to-have, but not essential.
Can we also add pretty
and eq
instances for Config
here?
Eq
I can add, but Pretty
isn't something we use very much in general in Plutarch.
This adds the ability to specify a log level as well as a tracing mode, both at compile time for scripts, and also when injecting traces into generated code. By doing this, we can log 'different amounts' at the same time (for example, an error code at info level, but a much bigger message at debug level).
Currently, we don't test this new functionality: to ensure CI passes, deprecation warnings have been disabled for the relevant test module, but this needs to be rewritten if this functionality turns out to be beneficial.