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

Mistaken usage of java.util Logger #1309

Closed
brodmo opened this issue Sep 20, 2023 · 1 comment · Fixed by #1411
Closed

Mistaken usage of java.util Logger #1309

brodmo opened this issue Sep 20, 2023 · 1 comment · Fixed by #1411
Labels
bug Issue/PR that involves a bug minor Minor issue/feature/contribution/change
Milestone
v5.0.0

Comments

@brodmo
Copy link
Contributor

brodmo commented Sep 20, 2023

From what I can gather, we use slf4j for logging. There are a couple of exceptions where the java.util Logger was mistakenly used. These usages should be removed. Additionally, it would be good if we could catch such cases using the CI pipeline in the future.
@brodmo brodmo changed the title Mistake usage of java.util Logger Mistaken usage of java.util Logger Sep 20, 2023
@tsaglam tsaglam added bug Issue/PR that involves a bug minor Minor issue/feature/contribution/change labels Sep 28, 2023
@tsaglam tsaglam added this to the v5.0.0 milestone Sep 28, 2023
@dfuchss dfuchss mentioned this issue Dec 3, 2023
Merged
@tsaglam
Copy link
Member

tsaglam commented Dec 5, 2023

Closed by #1411.

@tsaglam tsaglam closed this as completed Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issue/PR that involves a bug minor Minor issue/feature/contribution/change
Projects
None yet
Milestone
v5.0.0
Development

Successfully merging a pull request may close this issue.

Unify logger & Enforce Logger by Architecture Test jplag/JPlag
2 participants
@tsaglam @brodmo