Time |
Nick |
Message |
00:39 |
|
fluxflux joined #minetest-dev |
02:15 |
|
ShadowNinja joined #minetest-dev |
02:17 |
|
ircSparky joined #minetest-dev |
05:22 |
|
T4im joined #minetest-dev |
06:26 |
|
T4im joined #minetest-dev |
07:06 |
|
Foz joined #minetest-dev |
07:54 |
|
Foz joined #minetest-dev |
08:00 |
|
ShadowNinja joined #minetest-dev |
08:19 |
|
Foz joined #minetest-dev |
09:15 |
|
MTDiscord1 joined #minetest-dev |
09:19 |
|
mizux joined #minetest-dev |
09:28 |
|
calcul0n joined #minetest-dev |
10:03 |
|
Fixer joined #minetest-dev |
10:23 |
|
jas_ joined #minetest-dev |
10:32 |
|
proller joined #minetest-dev |
12:15 |
|
bootlegbilly joined #minetest-dev |
12:18 |
|
proller joined #minetest-dev |
12:30 |
|
absurb joined #minetest-dev |
13:50 |
|
bootlegbilly joined #minetest-dev |
13:58 |
rubenwardy |
Marked all action needed/WIP PRs that haven't been updated in a month as Possible Close |
14:03 |
|
Wuzzy joined #minetest-dev |
14:08 |
|
proller joined #minetest-dev |
14:33 |
|
bootlegbilly joined #minetest-dev |
15:36 |
|
paramat joined #minetest-dev |
15:39 |
paramat |
yes good idea. reducing the time limit for neglected PRs will help 'poke' the authors into action, a 6 month time limit was too long |
15:40 |
paramat |
merging game#2718 |
15:40 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/2718 -- Add icons for invisible nodes by Wuzzy2 |
15:52 |
paramat |
i suggest a maximum 3 month time limit for everything: neglected PRs, unsupported feature requests |
17:51 |
|
homthack joined #minetest-dev |
18:06 |
|
calcul0n_ joined #minetest-dev |
18:47 |
|
aldum joined #minetest-dev |
18:52 |
|
fluxflux joined #minetest-dev |
18:53 |
|
Fixer_ joined #minetest-dev |
18:59 |
|
Fixer joined #minetest-dev |
19:36 |
|
_Zaizen_ joined #minetest-dev |
19:36 |
|
Zughy joined #minetest-dev |
19:38 |
Zughy |
Hey there, sorry to bother. I was wondering if other core devs could share their perspective as v-rob and ruben did, when you're free of course: https://forum.minetest.net/viewtopic.php?f=3&t=25417 |
19:40 |
sfan5 |
there's not much to be said IMO |
19:40 |
sfan5 |
yes there's an issue but no there's no easy fix |
19:41 |
sfan5 |
regarding a concrete roadmap: maybe that'd be good but I don't know how well it'd work |
19:41 |
Jordach |
maybe change the reviewing process for something less overtly restrictive |
19:44 |
Jordach |
i only submit PRs for features for something i'm trying to develop |
19:46 |
rubenwardy |
I think the issue is more in allocation of time than the review process. A review process of "Check out the code, test it locally, inspect the code, comment" is solid |
19:51 |
Zughy |
sfan5, I was referring to the part about how you core devs feel about it, not about the existence of the issue |
19:53 |
sfan5 |
how am I supposed to "feel" about criticism? |
19:54 |
Zughy |
aaaa, not about what I said, about all the PRs, your expectations etc |
19:55 |
sfan5 |
.. |
19:56 |
sfan5 |
is "Minetest could be a lot better if the issues with development were sorted out and I had more time+motivation to work on features I care about" more like it? |
20:00 |
Krock |
TL;DR |
20:02 |
|
proller joined #minetest-dev |
22:12 |
|
proller joined #minetest-dev |
22:35 |
MTDiscord |
<Lone_Wolf> Can you not use a mod's texture folder to override textures anymore? |
22:36 |
MTDiscord |
<Lone_Wolf> Works in 5.3 but not 5.4 |
22:37 |
MTDiscord |
<Lone_Wolf> I'm using this mod with CTF https://github.com/MT-CTF/seasonal_halloween/tree/master/texture_changes |
22:37 |
Jordach |
holy hell there's an attachment bug that's sneakier than you think |
22:37 |
Jordach |
it involves EUJOR_CONTROL again |
22:38 |
Jordach |
set_bone_position properly sets armatures into local positioning based on updating their skeleton proper |
22:38 |
Jordach |
while set_attach rotating and positioning are globals |
22:38 |
Jordach |
and does not properly invoke EUJOR_CONTROL |
22:41 |
Jordach |
and while my PR fixes children visibility, it doesn't reset armatures proper |
23:27 |
|
fluxflux joined #minetest-dev |
23:42 |
|
homthack joined #minetest-dev |
23:58 |
MTDiscord |
<Lone_Wolf> Found the cause of my issue https://github.com/minetest/minetest/pull/9065 |