On Friday, May 22, 2015 at 12:06:05 PM UTC-7, David Rajchenbach-Teller wrote:
> On 22/05/15 04:06, Eric Rahm wrote:
> > After a rather thorough examination of usages of each I have come up with a
> > set that I believe would fit our needs:
> >
> > enum class LogLevel {
> > Disabled = 0, // Logging is disabled for this module
> > Error,
> > Warning,
> > Info,
> > Debug,
> > };
>
> I'll vote to have `Error` cause test failures unless explicitly whitelisted.
>
> >
>
>
> --
> David Rajchenbach-Teller, PhD
> Performance Team, Mozilla
This is an interesting proposal. To make this work we'd need to:
- Enable error level logging during test runs for all modules
- Actually output the log level.
- Do some automated analysis to figure out what needs to be whitelisted.
I'd suggest filing a bug to track this proposal (or perhaps split out a
separate thread).
I think it could fit into the idea of structured logging as well, but as Nick
pointed out that's some serious scope creep!
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
