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

Port shuttle sounds from RuPara #767

Merged
merged 11 commits into from
Nov 27, 2023
Merged

Port shuttle sounds from RuPara #767

merged 11 commits into from
Nov 27, 2023

Conversation

HelmsDeepState
Copy link

@HelmsDeepState HelmsDeepState commented Nov 25, 2023

Что этот PR делает

Добавляет микрошаттлам звук перелёта. Меняет звук эвакуационному шаттлу.
closes #324

Почему это хорошо для игры

Звук микрошаттлам меняется модульно, как сделать то же у эвакуационного не додумался. Посмотрел прошлые ПРы на изменение звука, путь меняли хардкодом xdd
Если критично, можно зарезать немодульные изменения и оставить звук микрошаттлам.

Тестирование

Trust me bro

Changelog

🆑
soundadd: Портировал звук шаттлу шахты и каторги. Портировал звук Харону
/:cl:

@github-actions github-actions bot added the 🔉 Звук Услада для ушей. label Nov 25, 2023
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Перенеси shuttle_sounds в модуль aesthetics_sounds, не нужно создавать для этого отдельный модуль

code/modules/shuttle/emergency.dm Outdated Show resolved Hide resolved
code/modules/shuttle/emergency.dm Outdated Show resolved Hide resolved
code/modules/shuttle/emergency.dm Outdated Show resolved Hide resolved
code/modules/shuttle/emergency.dm Outdated Show resolved Hide resolved
@HelmsDeepState HelmsDeepState requested a review from dj-34 November 26, 2023 13:44
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Зачем создавать отдельный файл SSshutles.dm? Засунь его код в shuttle.dm

@dj-34
Copy link
Collaborator

dj-34 commented Nov 27, 2023

не забывай resolve conversation под реквестами прожимать

@HelmsDeepState HelmsDeepState requested a review from dj-34 November 27, 2023 10:28
@dj-34
Copy link
Collaborator

dj-34 commented Nov 27, 2023

В следующий раз заливай ПР через отдельную ветку, а не с master'а
https://wiki.ss220.club/index.php/%D0%A0%D1%83%D0%BA%D0%BE%D0%B2%D0%BE%D0%B4%D1%81%D1%82%D0%B2%D0%BE_%D0%BF%D0%BE_%D1%80%D0%B0%D0%B7%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D0%BA%D0%B5

Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Модульно. Ок.
То что в мастере - не ок.

@dj-34 dj-34 requested a review from Legendaxe November 27, 2023 13:15
@AyIong AyIong merged commit 8691e60 into ss220club:master Nov 27, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 CL валиден 🔉 Звук Услада для ушей.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Перенести звуки полета шаттлов с РуПары
6 participants