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

Misc ApplicationError fixes #410

Merged

sleberknight merged 1 commit into main from fixes-to-ApplicationError

Sep 2, 2024

Member

sleberknight commented Sep 2, 2024

Remove the synchronized keyword from the two methods that are using the Lombok Synchronized annotation, since that defeats the purpose.
Remove VisibleForTesting from clearPersistentHostInformation because it is used by the ApplicationErrorExtension, which here is technically production code.


          Misc ApplicationError fixes

1b03239

* Remove the synchronized keyword from the two methods that are
  using the Lombok Synchronized annotation, since that defeats
  the purpose.
* Remove VisibleForTesting from clearPersistentHostInformation
  because it is used by the ApplicationErrorExtension, which
  here is technically production code.

sleberknight added the code cleanup label

sleberknight added this to the 3.0.0 milestone

sleberknight self-assigned this

sonarqubecloud bot commented Sep 2, 2024

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

sleberknight merged commit 3e7c7f8 into main

5 checks passed

sleberknight deleted the fixes-to-ApplicationError branch

September 2, 2024 20:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels