Time |
Nick |
Message |
01:31 |
|
PrairieWind joined #minetest-dev |
01:31 |
|
PrairieWind joined #minetest-dev |
04:00 |
|
MTDiscord joined #minetest-dev |
05:32 |
|
calcul0n joined #minetest-dev |
06:33 |
|
Fleckenstein joined #minetest-dev |
07:43 |
|
vampirefrog joined #minetest-dev |
08:04 |
nrz |
great job for the release , team 🙂 |
08:22 |
|
Warr1024 joined #minetest-dev |
09:36 |
|
Noisytoot joined #minetest-dev |
12:45 |
|
tekakutli joined #minetest-dev |
13:25 |
|
Desour joined #minetest-dev |
14:48 |
|
fluxionary_ joined #minetest-dev |
16:00 |
|
tekakutli joined #minetest-dev |
16:15 |
|
tekakutli joined #minetest-dev |
17:05 |
|
lhofhansl joined #minetest-dev |
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 |
17:36 |
|
Desour joined #minetest-dev |
18:26 |
|
PrairieWind joined #minetest-dev |
18:26 |
|
PrairieWind joined #minetest-dev |
18:52 |
|
olliy joined #minetest-dev |
19:29 |
|
PrairieWind joined #minetest-dev |
19:29 |
|
PrairieWind joined #minetest-dev |
20:05 |
|
proller joined #minetest-dev |
20:11 |
|
Desour joined #minetest-dev |
20:22 |
|
klbth joined #minetest-dev |
20:24 |
klbth |
rubenwardy: bad idea to ban IP adress ranges when you have a dynamic one |
20:43 |
|
klbth51 joined #minetest-dev |
20:48 |
|
vampirefrog joined #minetest-dev |
20:48 |
|
tekakutli joined #minetest-dev |
20:51 |
|
HuguesRoss3 left #minetest-dev |
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:16 |
|
fluxionary_ joined #minetest-dev |
21:32 |
|
moonmoon joined #minetest-dev |
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 |
22:36 |
|
MTDiscord1 joined #minetest-dev |