Skip to content

feature/teams - #29

Open
maxryan008 wants to merge 6 commits into
minecraft/1.21from
feature/teams
Open

maxryan008 wants to merge 6 commits into
minecraft/1.21from
feature/teams

Conversation

@maxryan008

Copy link
Copy Markdown
Member

Implement a team registration system for other mods to use aswell as a base default team system that uses vanilla minecraft's scoreboard team command system.

… for machine lib to use. Vanilla minecraft teams is the default however if a mod has a better team system and wishes to use it with machine lib they can now register their team system with machine lib and then machine libs config allows using that specific team system.
# Conflicts:
#	src/main/java/dev/galacticraft/machinelib/impl/config/MachineLibConfig.java

@Roelymole Roelymole 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.

This looks like a really powerful feature.

Comment thread src/main/java/dev/galacticraft/machinelib/impl/config/MachineLibConfig.java Outdated
Comment thread build.gradle.kts Outdated
# Conflicts:
#	src/main/java/dev/galacticraft/machinelib/impl/config/MachineLibConfig.java
Comment on lines +84 to +85
@Override @NotNull
public String teamSystem() {

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.

The convention in almost all of MachineLib is to put the annotation between public and String, so it is probably best to stick with that.

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.

2 participants