qa: checkstyle, pmd

[soloturn]

addressed pmd and checkstyle warnings:

• line too long
• logger not parameterized
• intellij warnings about formatting, e.g. @ReceiveEvent(components = {InventoryComponent.class}) replaced with @ReceiveEvent(components = InventoryComponent.class)
• unused private field

[jdrueckert]

Consider breaking lines at more readable points, e.g.

        BeforeItemRemovedFromInventory removeFrom = new BeforeItemRemovedFromInventory(instigator,
                InventoryUtils.getItemAt(from, slotFrom), slotFrom);

[jdrueckert]

While conventional commits are not mandatory within a PR (due to the fact that we're typically squash-merging), it does help people merging your PRs if you at least follow the convention in your PR titles

[soloturn]

While conventional commits are not mandatory within a PR (due to the fact that we're typically squash-merging), it does help people merging your PRs if you at least follow the convention in your PR titles

rebased on top of your pull request so your commits with the respective commit messages are there.

[jdrueckert]

can you please include in the PR description, which checkstyle and pmd warnings you addressed?
for one you already did it in your commit message (checkstyle, remove empty javadoc), but for the remaining changes it's unclear

[soloturn]

can you please include in the PR description, which checkstyle and pmd warnings you addressed? for one you already did it in your commit message (checkstyle, remove empty javadoc), but for the remaining changes it's unclear

updated pull request text, as well as commit message of the respective commit accordingly.