Skip to content
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

Cross-reference the node level manipulation functions #10633

Merged
merged 1 commit into from Dec 4, 2020

Conversation

Oblomov
Copy link
Contributor

@Oblomov Oblomov commented Nov 14, 2020

This can help developers find the correct functions to access and
manipulate the fluid level.

(When I was hacking around with fluids, I didn't find the relevant functions right off the bat, despite finding that param2 was the field to manipulate, so I was trying to come up with fancy ways to do the manipulation in Lua, before people on IRC pointed me to the right functions. By adding a reference to the functions I wish to spare other mod developers from missing them like I did.)

This can help developers find the correct functions to access and
manipulate the fluid level.
@sfan5 sfan5 added @ Documentation Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines labels Nov 14, 2020
Copy link
Member

@SmallJoker SmallJoker left a comment

Choose a reason for hiding this comment

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

Yes sure, why not.

@SmallJoker SmallJoker merged commit 08c9d1a into minetest:master Dec 4, 2020
HybridDog pushed a commit to HybridDog/minetest that referenced this pull request Dec 17, 2020
This can help developers find the correct functions to access and manipulate the fluid level.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Documentation One approval ✅ ◻️ Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants