Skip to content

Remove deprecated psr/log TestLogger#57

Merged
romantmb merged 1 commit into
mainfrom
roman-remove-testlogger
Apr 29, 2025
Merged

Remove deprecated psr/log TestLogger#57
romantmb merged 1 commit into
mainfrom
roman-remove-testlogger

Conversation

@romantmb
Copy link
Copy Markdown
Contributor

Replace deprecated TestLogger (removed from psr/log) with TestHandler + Logger from monolog/monolog repo

@romantmb romantmb marked this pull request as ready for review April 29, 2025 11:32
@romantmb romantmb requested a review from pepamartinec April 29, 2025 11:32
Copy link
Copy Markdown
Contributor

@pepamartinec pepamartinec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nejsem si jistej s tou upravou assertu - to jsem nikde nemusel delat. Jinak 👍

@romantmb romantmb merged commit 9e0244c into main Apr 29, 2025
1 check passed
@romantmb romantmb deleted the roman-remove-testlogger branch April 29, 2025 11:42
@romantmb
Copy link
Copy Markdown
Contributor Author

Nejsem si jistej s tou upravou assertu - to jsem nikde nemusel delat. Jinak 👍

@pepamartinec Storm mi tam prapodivně hintoval, že jsou tyhle helpery (hasInfo() atd.) bez argumentu. Nechtěl jsem se na tom zapéct, tak jsem to radši nahradil těmi ukecanějšími variantami.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants