-
Notifications
You must be signed in to change notification settings - Fork 37
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.
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
Add AI generated Javadoc #140
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments for the following classes are fine:
SplashProgress
DungeonHooks
CapabilityEnergy
(partially, I've left comments)IRenderHandler
(partially, I've left comments)MinecraftForgeClient
(partially, I've left comments)
I wouldn't rely on AI for generating javadocs especially in and around Minecraft and Forge code, as it wouldn't have enough context. As you can see, a lot of it are redundant and verbose, especially for methods that are self-explanatory.
I've kept some docs that does help clarify things, please amend and we'll look to pull this in.
src/main/java/net/minecraftforge/fml/client/SplashProgress.java
Outdated
Show resolved
Hide resolved
src/main/java/net/minecraftforge/fml/client/GuiSortingProblem.java
Outdated
Show resolved
Hide resolved
src/main/java/net/minecraftforge/fml/client/GuiSortingProblem.java
Outdated
Show resolved
Hide resolved
Ecdcaeb's suggestion Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
number 2! Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
number 3! Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
…m/Cleanroom into javadoc-thats-not-needed
src/main/java/net/minecraftforge/fml/client/GuiSortingProblem.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
src/main/java/net/minecraftforge/client/FluidContainerColorer.java
Outdated
Show resolved
Hide resolved
…java Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
@@ -26,14 +26,32 @@ | |||
|
|||
import javax.annotation.Nonnull; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FluidContainerColorer is not a api. No java docs is needed here.
@@ -26,8 +26,17 @@ | |||
import net.minecraftforge.fluids.FluidRegistry; | |||
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ForgeClientHandler is not a api. No java docs is needed here.
src/main/java/net/minecraftforge/client/MinecraftForgeClient.java
Outdated
Show resolved
Hide resolved
public static BlockRenderLayer getRenderLayer() | ||
{ | ||
return ForgeHooksClient.renderLayer.get(); | ||
} | ||
|
||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont delete it!
@@ -72,14 +69,6 @@ public static Locale getLocale() | |||
stencilBits.set(0,8); | |||
} | |||
|
|||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont delete it!
src/main/java/net/minecraftforge/fml/client/ExtendedServerListData.java
Outdated
Show resolved
Hide resolved
@@ -25,16 +25,35 @@ | |||
import java.io.File; | |||
import java.util.Map.Entry; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GuiDupesFound is not a API, java doc is not needed.
@@ -21,15 +21,33 @@ | |||
|
|||
import net.minecraft.network.datasync.DataSerializer; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for such complicated Java doc
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
…Data.java Co-authored-by: Hileb <107909747+Ecdcaeb@users.noreply.github.com>
This PR:
Why:
Notice: I have no idea how to test this! Could totally break everything! (Please teach me)