Time  Nick      Message
08:04 nrz       great job for the release , team 🙂
17:05 lhofhansl Merging #13385 in a few.
17:05 ShadowBot https://github.com/minetest/minetest/issues/13385 -- Allow large -z eye offsets by lhofhansl
17:05 lhofhansl Oops... I meant #13393.
17:05 ShadowBot https://github.com/minetest/minetest/issues/13393 -- Do not call updateDrawList, updateDrawListShadow, and touchMapBlocks in the same frame by lhofhansl
17:06 lhofhansl (not merging 13385)
17:10 lhofhansl done
17:11 lhofhansl Any comment on #13369 and #13314? Both are trivial.
17:11 ShadowBot https://github.com/minetest/minetest/issues/13369 -- Reverse Z-coordinate in 3rd person front view by lhofhansl
17:11 ShadowBot https://github.com/minetest/minetest/issues/13314 -- Add some instrumentation by lhofhansl
20:24 klbth     rubenwardy: bad idea to ban IP adress ranges when you have a dynamic one
21:01 Zughy[m]  I've checked all TurkeyMcMac's PRs and, except for #13005, no one has got any conflict with master. Could be a good idea to focus on reviewing his PRs before any conflict emerge? Instead of just closing them I mean
21:01 ShadowBot https://github.com/minetest/minetest/issues/13005 -- Fix crack with animation by TurkeyMcMac
21:02 Zughy[m]  PRs: https://github.com/minetest/minetest/pulls/TurkeyMcMac
21:02 Zughy[m]  he was a core dev, so one approval is enough
21:04 Desour    ^ only one approval would assume an implicit approval of him. AFAIK, we don't have implicit approvals
21:06 Zughy[m]  I mean, when a core dev makes a PR, the unwritten rule is that one approval is enough, so I don't see the difference in this case. If he's made the PR, it means he was sure of it
21:08 Desour    IIRC, the "one (non-self) approval is enough" rule only applies if the author merges, which is sadly not possible in this case
21:44 MTDiscord <Jonathon> Rather than getting bogged down in policy, would seem better to just look them over at least. As for just merging them (if ok) setting a precident, one would think it's not likely to be an issue due to hopefully circumstances not being met again