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

feat: add blood brother to Check Antagonists #877

Merged
merged 5 commits into from
Dec 19, 2023

Conversation

Gaxeer
Copy link
Collaborator

@Gaxeer Gaxeer commented Dec 19, 2023

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

Добавляет Blood Brother в Check Antagonists админ панель

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

Админ попросил

Изображения изменений

В коде

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

Скомпилировал, выдал антага, открыл манель

Changelog

🆑
add: Добавляет Blood Brother в Check Antagonists админ панель
/:cl:

@ss220bot ss220bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Dec 19, 2023
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.

Это пиздец. Лучше хардкодом в /datum/admins/proc/check_antagonists() это запиши.
Добавь туда просто:

// SS220 EDIT - START
		if(SSticker.mode.blood_brothers.len)
			dat += check_role_table("Blood Brothers", SSticker.mode.blood_brothers)

// SS220 EDIT - END

@Gaxeer Gaxeer requested a review from PhantornRU December 19, 2023 10:38
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.

Пока жду выноса /datum/mind/proc/memory_edit_blood_brother() в модуль.
Еще с дефайнами сомневаюсь.
Их хочется тоже в модуль, да и в принципе так и нужно, вместо раскидывания и поисков в будущем.

@Gaxeer Gaxeer requested a review from PhantornRU December 19, 2023 11:05
@Legendaxe Legendaxe merged commit 4fb87bd into ss220club:master Dec 19, 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.

4 participants