Skip to content

mild cleanup#68

Open
The4codeblocks wants to merge 6 commits intomt-mods:masterfrom
The4codeblocks:patch-5
Open

mild cleanup#68
The4codeblocks wants to merge 6 commits intomt-mods:masterfrom
The4codeblocks:patch-5

Conversation

@The4codeblocks
Copy link
Contributor

No description provided.

@The4codeblocks The4codeblocks marked this pull request as ready for review February 21, 2026 11:14
Copy link
Contributor

@wsor4035 wsor4035 left a comment

Choose a reason for hiding this comment

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

a bunch of line endings swapped pointlessly. functionally this pr breaks support on older versions, moves an entry around in a table, and fixes a few comments. not sure outside of the comment changes whats of value here

if type(minetest.get_game_info) == "function" then
gameid = minetest.get_game_info().id
--while minetest game derviates are not supported, we can still try to detect them
if minetest.get_modpath("default") then gameid = "minetest" end
Copy link
Contributor

Choose a reason for hiding this comment

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

moving this here breaks support for versions without this function

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if there is no function, won't it just run the other stuff?

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.

2 participants