Contribute
We welcome contributions from anyone for any feature related to Hypixel Skyblock. Please feel free to ask any question you may have.
For code style, try to follow the Fabric Development Guidelines. We will enforce TitleCase for classes, SCREAMING_SNAKE_CASE for static final fields except for specific exceptions, and camelCase for methods and all other fields.
We require Javadocs and comments for your classes and methods, except for getters/settings, boilerplate, or other simple methods. Also, try to include implementation details and document legal and illegal values for parameters and return values (nullability annotations, for instance). For examples of well-documented classes, see TooltipAdder, DungeonManager, ItemUtils, and RenderHelper.
Appropriate annotations are also welcome, such as @org.jetbrains.annotations.Nullable, @Language("RegExp"), or @Translatable.
Prefer fabric events over any mixins. Prefer mixin extras over invasive mixins such as redirects.
Remember to add configuration to your feature! Since Skyblocker 1.15, we use YACL as your config library. You need to add config fields to SkyblockerConfig and add an entry to the respective category, such as DungeonsCategory. You will also need to add the English localization to en_us.json. Additionally, we try to keep the config and category files (config class, category class) in the same order to minimize confusion. The English localisation file (at least the config section) is sorted alphabetically.
Please test your features thoroughly before submitting a PR. Screenshots or showcases would additionally accelerate the review process of your PR and are strongly suggested. Please note clearly in the PR if it has not been tested.
Unit tests are also strongly encouraged. Note that you can create unit tests for methods containing Minecraft classes. See ItemUtilsTest for an example.
Thank you for contributing to Skyblocker and hope you have fun!
