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.

Sign up for GitHub

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

Remove Logger abstraction #52

Closed
andrewazores opened this issue Dec 14, 2020 · 0 comments · Fixed by #348
Closed

Remove Logger abstraction #52

andrewazores opened this issue Dec 14, 2020 · 0 comments · Fixed by #348
Assignees
@mwangggg
Labels
feat New feature or request

Comments

@andrewazores
Copy link
Member

Logger is now an abstraction around SLF4J, which is itself a logging abstraction/facade. This is redundant and no longer useful, so our Logger definition should be removed and simply replaced with the SLF4J equivalent.
@andrewazores andrewazores mentioned this issue Dec 14, 2020
Closed
@andrewazores andrewazores added feat New feature or request and removed enhancement labels Jun 6, 2022
@andrewazores andrewazores mentioned this issue May 26, 2023
Merged
@andrewazores andrewazores moved this to Todo in 3.0.0 release Sep 22, 2023
@andrewazores andrewazores mentioned this issue Oct 2, 2023
Closed
@mwangggg mwangggg self-assigned this Feb 12, 2024
@mwangggg mwangggg mentioned this issue Feb 12, 2024
Merged
@andrewazores andrewazores moved this from Todo to In Progress in 3.0.0 release Feb 13, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in 3.0.0 release Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwangggg mwangggg

Labels
feat New feature or request
Projects
No open projects
3.0.0 release
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(logs): remove logger abstraction mwangggg/cryostat-core
2 participants
@andrewazores @mwangggg