-
-
Notifications
You must be signed in to change notification settings - Fork 446
Conversation
You should put yourself in the licensing header instead of openosrs if it’s your code If parts are from the removed files just add yourself in addition to openosrs |
I always thought this plugin needed a rewrite. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a checkstyle file you can import to intellij that will fix most of these code style issues, I highly recommend doing that, then using the reformat code right click option in intellij.
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanConfig.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanConfig.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanConfig.java
Outdated
Show resolved
Hide resolved
...client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanOverlay.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanPlugin.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanPlugin.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanPlugin.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanPlugin.java
Outdated
Show resolved
Hide resolved
...-client/src/main/java/net/runelite/client/plugins/lizardmanshaman/LizardmanShamanPlugin.java
Outdated
Show resolved
Hide resolved
Imported the checkstyle and made the suggested changes. Added name to licensing header. |
Addresses #2174