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

Refactor/simplify town command #7039

Merged
merged 8 commits into from
Nov 5, 2023
Merged

Conversation

LlmDl
Copy link
Member

@LlmDl LlmDl commented Oct 18, 2023

Description:

Refactor some of the more complicated methods in TownCommand, which is our only F graded class according to
https://www.codefactor.io/repository/github/townyadvanced/towny/file/master/Towny/src/main/java/com/palmergames/bukkit/towny/command/TownCommand.java

If you're looking at this you probably want to view each commit individually.


New Nodes/Commands/ConfigOptions:


Relevant Towny Issue ticket:


  • I have tested this pull request for defects on a server.

By making this pull request, I represent that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the TownyAdvanced organization has the copyright to use and modify my contribution under the Towny License for perpetuity.

@LlmDl LlmDl force-pushed the refactor/simplify_town_command branch from f750ae6 to 74d7395 Compare October 31, 2023 13:18
@LlmDl LlmDl force-pushed the refactor/simplify_town_command branch from 74d7395 to 14ba33e Compare October 31, 2023 13:19
@LlmDl LlmDl marked this pull request as ready for review October 31, 2023 13:20
@LlmDl LlmDl added this to the 0.100.1.0 milestone Oct 31, 2023
@LlmDl LlmDl merged commit df260e9 into master Nov 5, 2023
4 checks passed
@LlmDl LlmDl deleted the refactor/simplify_town_command branch November 5, 2023 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant