Steven/fabric-example-mod
1
0
Fork 0
Code Issues 38 Pull Requests 2 Projects Releases Wiki Activity

Configadd #82

Closed
yakyakyakyak wants to merge 4 commits from Configadd into master
pull from: Configadd
merge into: Steven:master
Conversation 3 Commits 4 Files Changed 6 +4
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 073f66402c - Show all commits

4
src/main/java/net/refsonyak/endgameaddons/Config.java
View File

@@ -2,4 +2,8 @@ public class Config {
/*Class that contains all configurable variables for quick'n'easy balance changes
YTG1234 commented 2021-02-07 10:47:46 +00:00 (Migrated from github.com)
Review
Copy Link

Please word more carefully and format the comment to either use // or have /* and */ on a separate line.

Please word more carefully and format the comment to either use `//` or have `/*` and `*/` on a separate line.
To use, simply create a class in here, give it some variables, import it into your
file. Call on it using Config.<yourclass>.<yourvariable> and voila!*/
public class dropRates {
YTG1234 commented 2021-02-07 10:50:42 +00:00 (Migrated from github.com)
Review
Copy Link

This is horrible naming conventions and classes should be static. You also do not need inner classes for everything. Static fields should be enough. Please also provide information about de/serialization and configuration screens.

This is horrible naming conventions and classes should be `static`. You also do not need inner classes for everything. Static fields should be enough. Please also provide information about de/serialization and configuration screens.
public static final double dropRate = 1;
}
}