Skip to content

Conversation

@ezenity
Copy link

@ezenity ezenity commented Mar 28, 2020

Willing to do as much JavaDocs as I can if you would like me to. I enjoy it and it actually helps me learn more about Java. I am not a beginner but I am also not a pro at Java but I'll do my best to keep it as generic and accurate as possible.

Copy link

@Sytm Sytm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some top tier JavaDocs in most places, but sometimes you are just explaining how it internally functions instead of what it actually is supposed to do.

I am gonna leave it open if you should actually JavaDoc private fields, I think it's weird but it kinda explains the fields better than without

/**
* Inventory close boolean. This field being a boolean, is the key to allowing the inventory to
* be closed or not.
*/
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This JavaDoc is just floating around with nothing attached to it

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the floating JavaDoc. Also added some more Javadocs along with clearing up the meaning behind what things are actually suppose to do. Will add more when I can, with whatever you have not added in v2.0.0

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants