Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
Default/trees: Add requirement of light level 13 for sapling growth
- Loading branch information
ef8b7e2
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.
Try to choose a clearer variable name like
light_level
for not requiring a comment aside.ef8b7e2
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.
I'm guessing you tried to align these comments with a tab width of four. Don't do that -- it breaks with every other tab width. Instead put it before. It doesn't even make sense this way because "matches grass ..." isn't a comment about "return false".
ef8b7e2
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.
Yes, 'll' was only used to match the farming mod.
I agree the comments are sloppy.
I may have to clean this up.
ef8b7e2
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.
👍 for readable variable names. I'm not keen on inline comments either.