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

Add basic_privs setting #3977

Closed
wants to merge 1 commit into from
Closed

Conversation

rubenwardy
Copy link
Member

No description provided.

@est31
Copy link
Contributor

est31 commented Apr 9, 2016

👍 I like this better than #3976

@rubenwardy
Copy link
Member Author

Much simpler, too. Is it worth documenting in lua_api.txt?

@est31
Copy link
Contributor

est31 commented Apr 9, 2016

You mean fixing the docs? Yes, you can do it.

@sofar
Copy link
Contributor

sofar commented Apr 9, 2016

Ahh yes, this is nice!

@PilzAdam
Copy link
Contributor

👍

This doesn't belong into lua_api.txt, but I don't care that much.

@est31
Copy link
Contributor

est31 commented Apr 10, 2016

Ah well, its okay if its not documented in lua_api.txt too. Sorry I was mistaken and believed that the table case wasn't documented at all.

@rubenwardy
Copy link
Member Author

I added a note there as that's one of the places I'd look before bothering to go into builtin/.

@PilzAdam
Copy link
Contributor

@rubenwardy the problem is that it's not part of the API. What if the setting changes in the future? lua_api.txt might become inconsistent with minetest.conf.

@paramat
Copy link
Contributor

paramat commented Apr 28, 2016

Will merge soon.

@paramat
Copy link
Contributor

paramat commented Apr 28, 2016

17bfe2f

@paramat paramat closed this Apr 28, 2016
@rubenwardy rubenwardy deleted the basic_privs_2 branch December 12, 2016 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants