Skip to content

Fix checkstyle issues

David Flor requested to merge xflord/STR-1148/checkstyle into main

Description

refactor: fix more checkstyle violations

  • manually fixed remaining issues
  • used regex replace for some constants
  • reordered overloaded methods next to each other
  • renamed more variables
  • adjusted lines that couldn't be automatically split
  • more fixes

How to test

You can run the checks and play around with the rules yourself.

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Other information

Read here

Related issues

TODO: Add relevant issue numbers in the following format (will be added automatically if the MR branch follows our branch naming conventions):

  • closes STR-XXX, closes STR-YYY for issues fully resolved by this MR, or
  • re STR-XXX, STR-YYY for issues only partially resolved by this MR

Re STR-1148

Edited by David Flor

Merge request reports