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

Горохострел (вместо револьверного дерева) #13498

Merged
merged 44 commits into from
Sep 8, 2024

Conversation

Danistans2
Copy link
Contributor

Описание изменений

Добавляет в ботанику горохостреловое дерево на замену револьверного дерева. С помощью вирусологии превращается в горохострел.

Почему и что этот ПР улучшит

ПР позволит пройти долгий и тернистый путь через ботанику и вирусологию, чтобы получить ГМО-пушку.

Глобально создаст больше возможностей для взаимодействия вирусолога с остальными и остальных с ним. Добавит в ботанику ещё одну прикольную фишку.

Авторство

Maleyvich — автор всего основного ПРа
mothershipzetafangirl — спрайты
Danistans — условие на вынимание магазина, микрофиксы значений

Чеинжлог

🆑

  • rscadd: Из игры удалено револьверное дерево, зато вместо него добавлен менее робастный и более логичный горохострел

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Sprites labels Aug 21, 2024
@TauKitty TauKitty requested a review from a team August 21, 2024 05:36
code/defines/obj/hydro.dm Outdated Show resolved Hide resolved
@Tap0r
Copy link
Contributor

Tap0r commented Aug 21, 2024

горохострел должен быть турелью

@Danistans2
Copy link
Contributor Author

Danistans2 commented Aug 21, 2024

горохострел должен быть турелью

Если будет мерж горохострела — будет горохотурель

Inb4: И звуки из PvZ

@ZER0-In
Copy link

ZER0-In commented Aug 21, 2024

горохострел должен быть турелью

Если будет мерж горохострела — будет горохотурель

Inb4: И звуки из PvZ

Если собирать турель и добавить вместо лазерной пушки горохострел = горохотурель?

@AzzyDreemurr13
Copy link
Contributor

Секунду, гляну что у тебя по тексту тут

@Danistans2
Copy link
Contributor Author

Danistans2 commented Aug 31, 2024

Выпилил весь перевод из Горохострела, его семян, патрон, etc.

Падежи не добавлял (и наоборот - удалил из тех мест, где замержил или вставил сам). Причина: Чтобы не было путаницы, когда буду заливать перевод гидропоники отдельным ПРом, чтобы не было вопросов, почему из всей гидропоники и патрон переведены только Горохострел и горошины. Потому что доверяю авторитетной позиции Людука в конце концов.

Перевода оружия с падежами в билде не обнаружил = приравнял уровень перевода к текущему для остальных видов патрон и оружия, ни больше, ни меньше. Скрин для сравнения уровня перевода:
image

Правки по проверкам (от Пикачушки) учел и внёс

@Danistans2
Copy link
Contributor Author

Вернул короче перевод как было, по комментам @Kiromaka

@AzzyDreemurr13

Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 left a comment

Choose a reason for hiding this comment

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

Жду последних правок и кину апрув. @Kiromaka также глянь пожалуйста, всё ли нормально кроме того, что я выделил.

code/defines/obj/hydro.dm Outdated Show resolved Hide resolved
code/modules/projectiles/ammunition/magazines.dm Outdated Show resolved Hide resolved
Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 left a comment

Choose a reason for hiding this comment

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

Вопросов более нет

@AzzyDreemurr13 AzzyDreemurr13 removed the Awaiting Review [Translation Dep] Ожидается ревью от департамента переводов label Sep 3, 2024
@Kiromaka
Copy link
Contributor

Kiromaka commented Sep 3, 2024

Мержите.

@volas volas merged commit ac24c14 into TauCetiStation:master Sep 8, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 8, 2024
@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/obsuzhdenie-pr-a-gorohostrel/43815/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.