New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable checkstyle #149

Merged

modmuss50 merged 2 commits into FabricMC:main from modmuss50:checkstyle

Jul 1, 2024

Member

modmuss50 commented Jun 19, 2024

Fix all checkstyle issues, all of them are from changes made in our fork
Disabled line lenght checkstyle rule, I think its tottaly silly.
Bumped the target language compatibily level to Java 8
- We already compiled to Java 8 bytecode, so why not allow using Java 8 language features.
- Slightly reduces our diff by removing the silly CompreUtils

modmuss50 added 2 commits

June 19, 2024 23:35


          Enable checkstyle

e9b4790


          Merge remote-tracking branch 'origin/main' into checkstyle

54add96

modmuss50 merged commit 2cd49e6 into FabricMC:main

2 checks passed

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

Labels

None yet