Revert "[Maps] Remove Tortuga" #1015
Conversation
This reverts commit d366ab5.
Обзор измененийWalkthroughВ данном обновлении представлен новый игровой карта "Tortuga" с подробной конфигурацией. Изменения включают добавление нового файла конфигурации карты, обновление тестов для интеграционного тестирования и модификацию пула карт по умолчанию. Несколько карт были закомментированы в стандартном пуле, а новая карта Tortuga добавлена с минимальным количеством игроков 10 и подробным описанием станции. Changes
Poem
Tip 🌐 Web search-backed reviews and chat
📜 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 (
|
github-actions
bot
added
S: Untriaged
Changes: Map
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/Maps/backmen_tortuga.yml (1)
10-14: Смешанные языки в шаблоне имени станцииШаблон имени использует русский текст, но префикс создателя указан на английском. Рекомендуется придерживаться одного языка.
Предлагаемое исправление:
- mapNameTemplate: '{0} Станция Тортуга {1}' - nameGenerator: - !type:NanotrasenNameGenerator - prefixCreator: '14' + mapNameTemplate: '{0} Тортуга {1}' + nameGenerator: + !type:NanotrasenNameGenerator + prefixCreator: 'Станция-14'
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
Content.IntegrationTests/Tests/PostMapInitTest.cs(1 hunks)Resources/Prototypes/_Backmen/Maps/Pool/backmenstandart.yml(1 hunks)Resources/Prototypes/_Backmen/Maps/backmen_tortuga.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (1)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (2)
Content.IntegrationTests/Tests/PostMapInitTest.cs (1)
83-83: Корректное добавление карты в тесты!Добавление "BackmenTortuga" в массив GameMaps обеспечивает тестирование загрузки карты.
Resources/Prototypes/_Backmen/Maps/backmen_tortuga.yml (1)
19-71: Проверить баланс рабочих местКонфигурация рабочих мест выглядит полной, но рекомендуется проверить:
- Соответствие минимальному количеству игроков (10)
- Баланс между отделами
- Закомментированные роли BKBPLATech и BKBPLAMED
✅ Verification successful
Баланс рабочих мест проверен и подтверждён
- Минимальное количество слотов (82) значительно превышает требуемый минимум в 10 игроков
- Баланс между отделами соблюдён и соответствует игровой механике
- Закомментированные роли BKBPLATech и BKBPLAMED корректно отключены и не влияют на конфигурацию
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверяем существование прототипов работ echo "Проверка существования прототипов работ..." for job in BKBPLATech BKBPLAMED; do if rg -q "type: job.*id: $job" Resources/Prototypes/; then echo "Прототип $job существует" else echo "ВНИМАНИЕ: Прототип $job не найден" fi doneLength of output: 453
| @@ -3,6 +3,7 @@ | |||
| maps: | |||
| - BackmenAspid | |||
| - BackmenKettle | |||
| #- BackmenTortuga | |||
There was a problem hiding this comment.
Карта Tortuga закомментирована в пуле карт!
Это противоречит цели PR по возврату карты Tortuga. Если PR предназначен для восстановления карты, она должна быть активна в пуле карт.
Предлагаемое исправление:
- #- BackmenTortuga
+ - BackmenTortuga📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| #- BackmenTortuga | |
| - BackmenTortuga |
| - type: StationEmergencyShuttle | ||
| emergencyShuttlePath: /Maps/Shuttles/NTES_Seal.yml |
There was a problem hiding this comment.
💡 Codebase verification
Некорректный путь к файлу шаттла эвакуации
Файл шаттла существует, но указан неверный путь. Необходимо исправить путь в конфигурации:
- Текущий путь:
/Maps/Shuttles/NTES_Seal.yml - Правильный путь:
/Resources/Maps/Shuttles/NTES_Seal.yml
🔗 Analysis chain
Проверить путь к шаттлу эвакуации
Необходимо убедиться, что файл /Maps/Shuttles/NTES_Seal.yml существует и совместим с картой.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверяем существование файла шаттла
if fd -t f "NTES_Seal.yml" Maps/Shuttles/; then
echo "Файл шаттла найден"
else
echo "ВНИМАНИЕ: Файл шаттла не найден"
fi
Length of output: 213
Script:
#!/bin/bash
# Поиск файла шаттла во всей репозитории
echo "Поиск файла NTES_Seal.yml..."
fd -t f "NTES_Seal.yml"
Length of output: 128
Rxup
changed the title
Reverts #1013
Summary by CodeRabbit
Новые функции
Изменения в конфигурации