Replace usages of colinodell/testlogger with BufferingLogger from symfony/error-handler, then remove dev dependency on testlogger

Created on 28 April 2023, over 1 year ago

Problem/Motivation

In πŸ“Œ Add colinodell/psr-test-logger to core's dev dependencies Fixed we added a new dev dependency on colinodell/testlogger
In πŸ“Œ Deprecate watchdog_exception Fixed we added some usages of it
I think we should use a different test logger that we already have from an existing dependency - \Symfony\Component\ErrorHandler\BufferingLogger

Pros

One less dev dependency to manage

Cons

The api is not as nice, you need to use cleanLogs and then filter things yourself

Steps to reproduce

Proposed resolution

Get folks to agree first - then replace the usages and remove the dependency

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Closed: won't fix

Version

10.1 ✨

Component
BaseΒ  β†’

Last updated 1 day ago

Created by

πŸ‡¦πŸ‡ΊAustralia larowlan πŸ‡¦πŸ‡ΊπŸ.au GMT+10

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024