Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Починка некоторых вещей, и добавление новогодних предметов #897

Merged
merged 20 commits into from
Dec 20, 2024

Conversation

sh4zik
Copy link
Contributor

@sh4zik sh4zik commented Dec 18, 2024

Описание PR

Добавление новогодних штучек, по заказу Pangaari, А также фиксы моего предыдущего ПРа

Почему / Баланс

Разнообразие сервера контентом, потому что, почему бы и нет, ну и починка этого разнообразного контента

Требования

  • Я прочитал(а) и следую Руководство по созданию пулл реквестов. Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
  • Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, или этот пулл реквест не требует демонстрации в игре

Чейнджлог

🆑 Sh4zik

  • add: Был добавлен снежок
  • add: Был добавлен огромный ящик, заполненным снегом
  • add: Были добавлены превосходные, зимние сани
  • add: Был добавлен ????? из слаймолюдов.
  • add: Было добавлено новогоднее настроение!
  • add: Была добавлены различные вариации кинжалов
  • add: Была добавлена возможность скрафтить новые виды вооружения - кинжалы
  • fix: Были починены крафты закалённых копей и заточек, а также их урон

@github-actions github-actions bot added S: Untriaged size/L Changes: Sprites Изменение спрайтов Changes: Localization Изменение локализации Changes: No C# and removed size/L S: Untriaged labels Dec 18, 2024
Copy link
Contributor

github-actions bot commented Dec 18, 2024

RSI Diff Bot; head commit e85ab9f merging into d0e6335
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/ADT/Objects/Fun/snowball.rsi

State Old New Status
snowball Added

Resources/Textures/ADT/Objects/Misc/slimehappines.rsi

State Old New Status
icon Added

Resources/Textures/ADT/Objects/Specific/snowdispenser.rsi

State Old New Status
icon Added

Resources/Textures/ADT/Objects/Weapons/Melee/wooden_sword.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/ADT/Structures/Specific/newyearkatalki.rsi

State Old New Status
newyearkatalka Added

Edit: diff updated after e85ab9f

@COTE-LAPYX
Copy link

Ну вот по хорошему стоило разбить этот пр на два разных, а именно первый пр - фиксы крафтов и добавление кинжалов
второй пр - добавление зимних штук из предложки.

Ибо если вдруг с кинжалами что-то будет не так, то из-за этого пострадает и дата релиза зимних вещей
Стоит совмещать ПРы если они связаны, ну и даже так, если они не зависят от друг друга, лучше делать мердж кью.

Это такой совет на будущее который уменьшает риски неприятных ситуаций.

@Schrodinger71
Copy link
Collaborator

Даа надо бы поднять вопрос о том, чтобы не создавать излишне громоздкие пулл-реквесты

@sh4zik
Copy link
Contributor Author

sh4zik commented Dec 19, 2024

Ну, Я глупый, Но на будущее запомню больше так не делать

Copy link
Contributor

@Darkiich Darkiich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ревью

@github-actions github-actions bot added the Status: Awaiting Changes Ожидание изменений label Dec 20, 2024
@sh4zik
Copy link
Contributor Author

sh4zik commented Dec 20, 2024

По какой-то, не понятной для меня причине гитхаб отказывается удалять пустые файлы, сам файл существует, но в нём ничего нет, как-то в целом странно гитхаб работает сегодня, но всё же

@Darkiich
Copy link
Contributor

По какой-то, не понятной для меня причине гитхаб отказывается удалять пустые файлы, сам файл существует, но в нём ничего нет, как-то в целом странно гитхаб работает сегодня, но всё же

Мгм, потом не забудь удалить через VSC

Copy link
Contributor

@Darkiich Darkiich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Боже, ты красаууучик

@github-actions github-actions bot removed the Status: Awaiting Changes Ожидание изменений label Dec 20, 2024
@Darkiich Darkiich merged commit 4beb9ea into master Dec 20, 2024
18 checks passed
@Darkiich Darkiich deleted the fixesandnewyear branch December 20, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: No C# Changes: Sprites Изменение спрайтов size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants