Skip to content

DeltaAndAstraUpdate - #1505

Merged
Rxup merged 6 commits into
Rxup:masterfrom
123Krab123:DeltaAndAstraUpdate
Aug 25, 2026
Merged

DeltaAndAstraUpdate#1505
Rxup merged 6 commits into
Rxup:masterfrom
123Krab123:DeltaAndAstraUpdate

Conversation

@123Krab123

@123Krab123 123Krab123 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

About the PR

Заремапили Астру и Дельту, закинули в маппул, усе

Why / Balance

Technical details

Test plan

Media

Requirements

  • I have read and am following the Pull Request and Changelog Guidelines.
  • I have tested this pull request and written instructions on how to test it
  • I have added media to this PR or it does not require an in-game showcase.

Breaking changes

Changelog

Summary by CodeRabbit

  • Новые возможности
    • В пул доступных карт добавлены станции PNS Astra и BM Delta.
    • Для обеих карт установлен диапазон вместимости от 15 до 60 игроков.
    • Для PNS Astra настроены доступные должности, роли Backmen и цели станции.
  • Изменения
    • Карта Delta переименована в BM Delta и получила новый шаблон названия станции.
    • Для карт настроены аварийные шаттлы и автоматическая генерация названий.
    • Из списка доступных должностей BM Delta удалена роль пилота.

@123Krab123
123Krab123 requested a review from Rxup as a code owner August 24, 2026 14:06
@github-actions github-actions Bot added S: Untriaged size/XL Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping. No C# Нету кода. and removed S: Untriaged size/XL labels Aug 24, 2026
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 33068e70-bb84-4efc-9862-accb84524550

📥 Commits

Reviewing files that changed from the base of the PR and between 1e13973 and 075e7aa.

📒 Files selected for processing (3)
  • Resources/Maps/Backmen/backmen_astra.yml
  • Resources/Maps/Backmen/backmen_delta.yml
  • Resources/Prototypes/_Backmen/Maps/backmen_delta.yml
💤 Files with no reviewable changes (1)
  • Resources/Prototypes/_Backmen/Maps/backmen_delta.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

Добавлена конфигурация карты BackmenAstra. Обновлена конфигурация BackmenDelta. Обе карты добавлены в BackmenDefaultMapPool с диапазоном вместимости 15–60.

Changes

Карты Backmen

Layer / File(s) Summary
Конфигурации карт
Resources/Prototypes/_Backmen/Maps/backmen_astra.yml, Resources/Prototypes/_Backmen/Maps/backmen_delta.yml
Добавлена полная конфигурация BackmenAstra. Для BackmenDelta обновлены имя, путь, диапазон игроков, шаблон имени станции, prefixCreator и список доступных профессий.
Регистрация карт в пуле
Resources/Prototypes/_Backmen/Maps/Pool/backmenstandart.yml
В BackmenDefaultMapPool добавлены BackmenAstra и BackmenDelta с ограничением вместимости 15-60.

Estimated code review effort: 2 (Simple) | ~10 минут

Merge Risk: ⚪ Minimal · up to 075e7

This PR updates and adds maps to the map pool; no actionable merge-blocking risk remains in the supplied evidence, so it is merge-ready after normal checks and review.

Suggested reviewers: rxup

Poem

Кролик проверил карты в ночи,
Astra и Delta добавил в пулы.
Игрокам открылись новые пути,
Лимиты стоят — 15–60.
YAML шуршит в морковной пыли.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Заголовок «DeltaAndAstraUpdate» соответствует основному изменению: обновлению карт Delta и Astra и добавлению их в пул карт. Заголовок краткий и связан с содержанием изменений, но формат CamelCase сни…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

Заголовок «DeltaAndAstraUpdate» соответствует основному изменению: обновлению карт Delta и Astra и добавлению их в пул карт. Заголовок краткий и связан с содержанием изменений, но формат CamelCase снижает читаемость.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ 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.

@123Krab123

Copy link
Copy Markdown
Contributor Author

кпц, че за проверки страшные

@123Krab123

Copy link
Copy Markdown
Contributor Author

Хуйня какая-то, у меня ласт версия локалки и форк синхронизирован. Че он руинит тест с культистами непонятно

Error: /home/runner/work/space-station-14/space-station-14/Content.IntegrationTests/Tests/Backmen/Flesh/FleshCultistDevourTest.cs(129,66): error CS1503: Argument 2: cannot convert from 'Robust.Shared.GameObjects.TransformComponent' to 'Robust.Shared.GameObjects.MetaDataComponent?' [/home/runner/work/space-station-14/space-station-14/Content.IntegrationTests/Content.IntegrationTests.csproj]

@123Krab123

Copy link
Copy Markdown
Contributor Author

Ну вобщем не опять, а снова.
Рхуп когда будешь проверять, если есть мысли, отпиши сюдой

@Rxup
Rxup merged commit 68ecea8 into Rxup:master Aug 25, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: Map Can be reviewed or fixed by people who are knowledgeable with mapping. No C# Нету кода. size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants