Skip to content
This repository was archived by the owner on Oct 1, 2025. It is now read-only.

Removing libs, adding dependencies, client language check and commets translation to English#1

Open
mrTeroblaze wants to merge 58 commits into
masterfrom
feature/remove-libs-deps-lang-check-comms-translate
Open

Removing libs, adding dependencies, client language check and commets translation to English#1
mrTeroblaze wants to merge 58 commits into
masterfrom
feature/remove-libs-deps-lang-check-comms-translate

Conversation

@mrTeroblaze

Copy link
Copy Markdown
Collaborator

Removed jars from libs folder Added Gregtech in dependencies Added client language check, so mod wont won't do anything when en_US is selected
Translated all comments in English.

@Dream-Master
Dream-Master requested a review from a team September 4, 2025 18:09

@Taskeren Taskeren left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I didn't look into it too much.
Just found something can be improved.

try {
FMLInterModComms
.sendMessage("Waila", "register", "com.teroblaze.gtitemuntranslator.waila.WailaRegister.register");
System.out.println("[" + NAME + "] Waila integration registered.");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

consider using loggers

if (commandManager instanceof ServerCommandManager) {
((ServerCommandManager) commandManager).registerCommand(new CommandGTIP());
System.out.println("[" + NAME + "] /gtip command registered.");
((ServerCommandManager) commandManager).registerCommand(new CommandTIPP());

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Commands should be registered in FMLServerStartingEvent, using its #registerServerCommand(ICommand).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants