Skip to content

[REMOVE] BloodBrother#2652

Open
CrimeMoot wants to merge 3 commits intoAdventureTimeSS14:masterfrom
CrimeMoot:remove_blood_brother
Open

[REMOVE] BloodBrother#2652
CrimeMoot wants to merge 3 commits intoAdventureTimeSS14:masterfrom
CrimeMoot:remove_blood_brother

Conversation

@CrimeMoot
Copy link
Contributor

Описание PR

Кровные братья — всё

Техническая информация

  • Изменения были протестированы на локальном сервере, и всё работает отлично.
  • PR закончен и требует просмотра изменений.

Чейнджлог

🆑 CrimeMoot

  • remove: Синдикат признали кровных братьев через-чур безбашенными и неконтролируемыми, по этому их списали и больше на операциях против НТ они не выступают.

@github-actions github-actions bot added size/M Status: Needs Review Changes: Sprites Изменение спрайтов and removed Status: Needs Review labels Mar 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2026

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

Resources/Textures/ADT/Interface/Misc/job_icons.rsi

State Old New Status
BloodBrother Removed
BloodBrotherLead Removed

Edit: diff updated after c7b6964

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e0681f35-c684-4482-85f1-5da4031d1e37

📥 Commits

Reviewing files that changed from the base of the PR and between 85aa209 and c7b6964.

📒 Files selected for processing (2)
  • Resources/Prototypes/ADT/GameRules/roundstart.yml
  • Resources/Prototypes/ADT/Objectives/objectiveGroups.yml
💤 Files with no reviewable changes (2)
  • Resources/Prototypes/ADT/Objectives/objectiveGroups.yml
  • Resources/Prototypes/ADT/GameRules/roundstart.yml

Walkthrough

Удалена реализация механики BloodBrothers: удалены клиентские и серверные системы, компоненты BloodBrotherComponent и BloodBrotherLeaderComponent, серверная логика конверсии, роль/мэйнд-компонент для BloodBrother, интеграции с Changeling и Heretic, все прототипы ролей/фракций/статус-иконок и соответствующие текстуры/конфигурации правил раунда. Код, который подписывался на события, генерировал брифинги, назначал фракции или вычислял вклад в ритуалы/эволюцию на основе ConvertedCount, удалён.

Possibly related PRs

Suggested labels

Changes: Localization, Status: Needs Review, S: Untriaged

Suggested reviewers

  • M1and1B
  • KashRas2
  • Ratyyy
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed Заголовок PR чётко и лаконично описывает основное изменение — удаление функционала BloodBrother. Соответствует сути всего набора изменений.
Description check ✅ Passed Описание PR связано с изменениями: объясняет причину удаления функционала BloodBrother и содержит техническую информацию и записи об изменениях.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@CrimeMoot CrimeMoot marked this pull request as draft March 23, 2026 19:46
@CrimeMoot CrimeMoot marked this pull request as ready for review March 23, 2026 19:46
@CrimeMoot CrimeMoot marked this pull request as draft March 23, 2026 22:13
@CrimeMoot CrimeMoot marked this pull request as ready for review March 23, 2026 22:13
@kerfus-owo
Copy link
Contributor

Герой... о7

@Unlumy
Copy link
Collaborator

Unlumy commented Mar 24, 2026

Да их бы переделать бы
Под 2 игрока с равными правами становятся братьями.

@CrimeMoot
Copy link
Contributor Author

Да их бы переделать бы
Под 2 игрока с равными правами становятся братьями.

Мне удалить попросили. Их дважды меняли как видишь толку 0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: Sprites Изменение спрайтов size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants