Skip to content

Add maven plugin to have ability run & debug towny as a mvn task#8182

Open
NeumimTo wants to merge 1 commit into
TownyAdvanced:masterfrom
NeumimTo:mvn-task
Open

Add maven plugin to have ability run & debug towny as a mvn task#8182
NeumimTo wants to merge 1 commit into
TownyAdvanced:masterfrom
NeumimTo:mvn-task

Conversation

@NeumimTo

@NeumimTo NeumimTo commented May 1, 2026

Copy link
Copy Markdown

Description:

Add ability to run and debug mc server from within a maven task


New Nodes/Commands/ConfigOptions:

xxx


Relevant Towny Issue ticket:

xxx


  • [ x] 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 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a lot of unneeded changes as far as I can tell, please review the changes made to whitespace to reduce the diff.

@NeumimTo

NeumimTo commented May 1, 2026

Copy link
Copy Markdown
Author

There are like no changes at all, just added one plugin, remaining changed lines is just fix of wrong indentation

@LlmDl

LlmDl commented May 2, 2026

Copy link
Copy Markdown
Member

There are like no changes at all, just added one plugin, remaining changed lines is just fix of wrong indentation

I see that now, can you still reduce the diff down to just what is required for this PR. Formatting changes can be done in a separate PR.

@Warriorrrr

Copy link
Copy Markdown
Member

If package relocations could also be disabled when the run task is used as described in that comment then that would be very nice

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants