minetest_game
minetest_game copied to clipboard
Farming: Mandatory definition fields, reformat documentation and code
Some additional changes: Hoes only work if the node is pointed to from above. Test for nil player in the hoe on_use function. Properly test if the creative mod is installed. Hoe and plant description, inventory_image, etc. must be specified (when registering). Nodes with the field group must have all soil values in the node definition. Field nodes do not loose their param2 value (e.g. rotation) when they dry or get wet, or when hoed. The node above the hoed one is not checked for protection because that node is not changed; similarly, the node under a seed is not tested for protection.
~Why is the cleanup in game_api related to doors included with this?
Shouldn't that go into a separate PR?
Or are you going to turn this into a general api cleanup PR?~
Sorry, having a bad day... just noticed the title mentions addtl changes.
The door related changes are only whitespace removals from line ends; my text editor does this automatically.
The three backticks are GitHub Favored only, whereas indenting is an "Official" notation. To support Markdown readers aside GitHub's web render, I'd like to not change the current style.
To support Markdown readers aside GitHub's web render, I'd like to not change the current style.
I've changed it back. I hope that having the comments in a separate line makes it easier to read than comments at line ends.
@HybridDog you can replace lbm action
action = function(pos, node)
tick_again(pos)
end,
on this:
action = tick_again
Thank you for your work!
@HybridDog, it seems rebase is required...
Stale, closing.