Mikita Wiśniewski rudzik8
  • pos: (26.8, 8.3, 33.1)
  • https://codeberg.org/rudzik8
  • Siberian 15yo Minetest player and modder, MineClone 2 contributor

    #nowar

  • Joined on 2022-02-18
rudzik8 pushed to mcl_chests_redo at VoxeLibre/VoxeLibre 2024-06-21 09:33:51 +02:00
9bed2c112a Clean-up ITEMS/mcl_chests code, part 2
rudzik8 commented on pull request VoxeLibre/VoxeLibre#4450 2024-06-21 06:07:08 +02:00
Fix extra 'Stair' in Polished Blackstone Brick nodes

It is non-plural in the locales.

rudzik8 suggested changes for VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:23 +02:00
Hudbars update

Good job! I think the work you've done is pretty good, although not perfect.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:22 +02:00
Hudbars update

Typo. layer_id

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:22 +02:00
Hudbars update

Top comment is wrong. 0 ~= nil. You are specifically returning 0 in this function.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:22 +02:00
Hudbars update

This line could really benefit from line breaks and spaces.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:22 +02:00
Hudbars update

Move all these settings assignments somewhere so that you aren't constantly writing vl_hudbars.settings.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4424 2024-06-20 17:01:22 +02:00
Hudbars update

Huh? XD

rudzik8 pushed to mcl_chests_redo at VoxeLibre/VoxeLibre 2024-06-20 15:09:00 +02:00
c78f38ad26 Clean-up ITEMS/mcl_chests code, part 1
rudzik8 created branch mcl_chests_redo in VoxeLibre/VoxeLibre 2024-06-20 15:08:59 +02:00
rudzik8 approved VoxeLibre/VoxeLibre#4338 2024-06-19 16:09:48 +02:00
mcl_weather sky color refactor

The clouds are still not visible when you're underwater, but when you just leave it, having no jitter appear is pleasant. +1.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:43:20 +02:00
mcl_weather sky color refactor

Why would you do it? nil acts as false (and not nil as true) either way, this is just confusing.

rudzik8 suggested changes for VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:36 +02:00
mcl_weather sky color refactor

The issue I was experiencing is now gone, so +1 for that. In these comments, I give you another +1 for a little something, so in total you have already earned 3 imaginary points from this PR. You're welcome. :D

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:36 +02:00
mcl_weather sky color refactor

I like this trick. You did an if statement, but returned it in the end just to keep it at one less tab. Using else here would save you the later comment about no weather applying, but I'll give you a +1 as this is nothing critical, just a neat detail.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:35 +02:00
mcl_weather sky color refactor

NIGHT_VISION_RATIO is not defined here, as it is still assigned locally at skycolor.lua.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:35 +02:00
mcl_weather sky color refactor

Why don't you just assign it at the beginning? This is really unnatural.

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:35 +02:00
mcl_weather sky color refactor

Line break

rudzik8 commented on pull request VoxeLibre/VoxeLibre#4338 2024-06-19 15:17:35 +02:00
mcl_weather sky color refactor

Don't forget to comment this out.

rudzik8 approved VoxeLibre/VoxeLibre#4317 2024-06-17 07:39:37 +02:00
Changes on slab placement checks

At first, I was worried about the protection check part getting removed, but after testing this with Areas and Protection Redo, this seems fine.