Time Nick Message 11:56 Zughy[m] celeron55: https://irc.minetest.net/minetest-dev/2023-05-01#i_6080728 15:55 imi hi, as far as I can tell I fixed the issues found in code review: https://github.com/minetest/minetest_game/compare/master...imre84:minetest_game:master what do I do now? Do I need to squash? Do I need to open a new pull request? 16:12 MTDiscord You don't need to squash or open a separate PR. 16:38 Krock usually PRs are squashed on rebase-merge. there's a convenient button for maintainers that does that, so no need to worry. 16:40 Krock about the slot count: I suppose that could be solved by using a scroll container, initialize a customizable amount of slots, fill them up and trim to the amount of occupied slots 16:40 Krock although that might already be an overengineered approach 17:02 imi currently I use a variable sized grid, with the minimum of 8*4 and if there are more items in the bones the grid grows