-
Notifications
You must be signed in to change notification settings - Fork 154
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
Box Backmen remap #1062
Box Backmen remap #1062
Conversation
""" WalkthroughВ конфигурационном файле Changes
Sequence Diagram(s)sequenceDiagram
participant Server as Игровой сервер
participant MapConfig as BackmenBox Config
participant Station as Boxstation
participant Shuttle as Конфигурация шаттла
participant JobSystem as Система распределения ролей
Server->>MapConfig: Загружает конфигурацию карты
MapConfig->>Station: Инициализация станции (Boxstation)
Station->>MapConfig: Настройка имени станции (StationNameSetup)
Station->>Shuttle: Настройка аварийного шаттла (StationEmergencyShuttle)
Station->>JobSystem: Регистрация рабочих ролей (StationJobs)
JobSystem-->>Station: Подтверждение настроек ролей
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 300000ms (2)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/Maps/backmen_box.yml (1)
17-72
: Определение вакансий и ролей.
РазделStationJobs
с перечнем доступных вакансий детально описывает позиции для различных секторов (сервис, инженерия, медицина, наука, безопасность, снабжение, гражданские, центком и силконы).
Обратите внимание на следующие моменты:
- Значение для
Passenger
задано как[ -1, -1 ]
. Проверьте, что применение отрицательных значений соответствует ожидаемому поведению (например, для отключения вакансии или указания неограниченного числа позиций).- Диапазоны, заданные в формате
[min, max]
, должны быть корректно обработаны игровой логикой.- Комментарии (например,
# Corvax
,# backmen-Brigmedic
,# backmen-Deliveryman
) полезны для идентификации изменений, но рекомендуется обеспечить их дополнительную документацию для понимания всей команды.
В целом, определение должностей выглядит исчерпывающим и аккуратно структурированным.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Maps/backmen_box.yml
(1 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (4)
Resources/Prototypes/_Backmen/Maps/backmen_box.yml (4)
1-5
: Заголовок и базовая конфигурация карты.
Файл начинается с определения типа карты, её идентификатора, имени, пути к файлу и минимального количества игроков. Все поля выглядят корректно. Обратите внимание, что использование тире (- type: gameMap
) подразумевает, что конфигурация представлена в виде списка. Убедитесь, что это соответствует общей схеме конфигурационных файлов вашего проекта.
6-8
: Определение станции.
Разделstations
с ключомBoxstation
и указанием прототипаStandardNanotrasenStation
корректно описывает используемую станцию. Проверьте, что прототип доступен и правильно реализован в проекте.
9-14
: Компонент StationNameSetup.
Конфигурация компонента для настройки имени станции выглядит аккуратно: задан шаблон имени (mapNameTemplate
) и использован генератор имен с пользовательским тегом!type:NanotrasenNameGenerator
. Убедитесь, что данный тип значения поддерживается вашим YAML-парсером и логика генерации работает корректно.
15-16
: Компонент аварийного шаттла.
Указан тип компонентаStationEmergencyShuttle
с корректно прописанным путём/Maps/Shuttles/emergency_box.yml
. Рекомендуется проверить, что файл конфигурации аварийного шаттла существует по указанному пути.
Ненавижу... |
@Agronomist-NN пофикси CL а не то насрёшь в игру чейнджлогом |
Убрал |
Кривое говно этот ваш маппинг... |
Описание PR
Данный PR это пересмотр стиля карты Box и другие мелкие изменения. Он добавляет карту Box в директорию Backmen и не влияет на карту основного билда.
Тип PR
🆑
Summary by CodeRabbit
Обновленные заметки о релизе