The Wayback Machine - https://web.archive.org/web/20220617004445/https://github.com/nextcloud/server/issues/32127
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Meta]: Port away from ILogger #32127

Open
2 of 13 tasks
CarlSchwan opened this issue Apr 25, 2022 · 2 comments
Open
2 of 13 tasks

[Meta]: Port away from ILogger #32127

CarlSchwan opened this issue Apr 25, 2022 · 2 comments
Labels
2. developing good first issue technical debt

Comments

@CarlSchwan
Copy link
Member

@CarlSchwan CarlSchwan commented Apr 25, 2022 •
edited by come-nc

Current pull requests

Current remaining modules to port in server

  • lib/{private,public}/Files
  • federation
  • encryption
  • federatedfilesharing
  • files_sharing
  • files_trashbin
  • files_external
  • ....

Current remaining shipped app to port

  • ....

Current issues

  • The background job interface exposes ILogger in its public API. We need to deprecate this
  • In some places, like the updater. It's helpful to be able to write ILogger:DEBUG instead of 2. We need to look if upstream psr would be interested to add enums for the log levels.
@CarlSchwan CarlSchwan added 2. developing technical debt good first issue labels Apr 25, 2022
@come-nc
Copy link
Contributor

@come-nc come-nc commented Apr 26, 2022 •
edited

@come-nc
Copy link
Contributor

@come-nc come-nc commented Apr 26, 2022 •
edited

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing good first issue technical debt
2 participants
@CarlSchwan @come-nc