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

PayloadUtil: simple sonarlint clean-up #496

Merged

Conversation

sambish5
Copy link
Collaborator

@sambish5 sambish5 commented Jun 5, 2023

cleaned up logging statements in PayloadUtil. Sonarlint also complains to use AbstractJDOMUtil instead of JDOMUtil, but figured I wouldn't change that unless specified to do so

@sambish5 sambish5 added the tech-debt Low-impact cleanup and upkeep label Jun 5, 2023
@jpdahlke jpdahlke added this to the v7.19.0 milestone Jun 5, 2023
fbruton
fbruton previously requested changes Jun 5, 2023
src/main/java/emissary/util/PayloadUtil.java Outdated Show resolved Hide resolved
src/main/java/emissary/util/PayloadUtil.java Outdated Show resolved Hide resolved
@sambish5 sambish5 requested a review from fbruton June 6, 2023 14:09
@jpdahlke jpdahlke self-requested a review June 7, 2023 13:32
@jpdahlke jpdahlke merged commit 23a7a44 into NationalSecurityAgency:master Jun 7, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Low-impact cleanup and upkeep
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants