Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Sep 14, 2024

Summary

This removes all public parts of the ILogger logging functions which were deprecated for quite some time.
What is kept are the log levels which are un-deprecated, because even if we now support PSR Log levels we need this for working with our log file handling.

Checklist

@susnux susnux added 3. to review Waiting for reviews technical debt ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Sep 14, 2024
@susnux susnux added this to the Nextcloud 31 milestone Sep 14, 2024
Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

Awesome

@susnux susnux force-pushed the chore/remove-ilogger branch 3 times, most recently from f76cb32 to a117646 Compare September 14, 2024 23:53
@susnux susnux requested a review from come-nc September 14, 2024 23:53
@susnux susnux force-pushed the chore/remove-ilogger branch from a117646 to 2c5bea4 Compare September 14, 2024 23:54
@susnux susnux force-pushed the chore/remove-ilogger branch from 47eb808 to d24ffc8 Compare September 15, 2024 11:17
susnux and others added 9 commits September 19, 2024 00:32
Signed-off-by: Ferdinand Thiessen <[email protected]>
Co-authored-by: Ferdinand Thiessen <[email protected]>
Co-authored-by: Côme Chilliet <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the chore/remove-ilogger branch from d24ffc8 to 5cce140 Compare September 18, 2024 22:37
@susnux susnux requested a review from come-nc September 18, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants