Ответвление карт Saltern, Marathon, Aspid от ADT и их исправления - #375
Ответвление карт Saltern, Marathon, Aspid от ADT и их исправления#375RedSpyy wants to merge 5 commits into
Conversation
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughОбзорОбновлена конфигурация пула карт по умолчанию для замены ссылок ADT на новые прототипы карт ENT, и добавлены три новых прототипа игровых карт (Aspid, Marathon, Saltern) с определением параметров населения и конфигурацией должностей станций. Изменения
Оценка затрат на проверку кода🎯 3 (Moderate) | ⏱️ ~25 minutes Предлагаемые метки
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Review rate limit: 0/1 reviews remaining, refill in 13 minutes and 56 seconds.Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
Resources/Prototypes/Maps/Pools/default.yml (1)
27-34: Актуализируйте комментарии диапазонов Midpop/Lowpop для читаемости.На Line 27, Line 33 и Line 34 замена ID корректная, но текстовые диапазоны секций (
35-60,20-35) теперь неочевидны относительно новыхminPlayers/maxPlayersуENT_*. Лучше синхронизировать комментарии с текущей логикой отбора, чтобы не путать следующих мейнтейнеров.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: fce0b688-336b-4639-b94b-affd0bc2ceb0
📒 Files selected for processing (7)
Resources/Maps/_Ganimed/Stations/ent_aspid.ymlResources/Maps/_Ganimed/Stations/ent_marathon.ymlResources/Maps/_Ganimed/Stations/ent_saltern.ymlResources/Prototypes/Maps/Pools/default.ymlResources/Prototypes/_Ganimed/Maps/ent_aspid_proto.ymlResources/Prototypes/_Ganimed/Maps/ent_marathon_proto.ymlResources/Prototypes/_Ganimed/Maps/ent_saltern_proto.yml
📜 Review details
🔇 Additional comments (3)
Resources/Prototypes/_Ganimed/Maps/ent_saltern_proto.yml (1)
1-20: Хорошее выделение карты в отдельныйgameMap-прототип.Структура прототипа (
id,mapPath,stations, shuttle-компоненты) выглядит целостно и соответствует цели форка карты без правок ADT-оригинала.Resources/Prototypes/_Ganimed/Maps/ent_marathon_proto.yml (1)
1-82: Изменение выглядит корректно и консистентно с остальными ENT-картами.Прототип
ENT_Marathonоформлен полноценно: структура станции, шаты и распределение ролей заданы в ожидаемом формате.Resources/Prototypes/_Ganimed/Maps/ent_aspid_proto.yml (1)
1-82: Хорошая декомпозиция Aspid в отдельный прототип.Конфиг читаемый, полный и соответствует заявленной стратегии «форк с собственным ID без модификации ADT-оригинала».
|
Для турелей замаплены и подключены панели? Я напомню, что турели ДОЛЖНЫ быть на спутнике (корридоры), в ядре и в комнате с консолью законов. Все эти три "категории" должны иметь свои панели управления турелями. |
There was a problem hiding this comment.
Все названия файлов "ent_{mapname}" и "ent_{mapname}_proto" поменять, оставив только название карты "{mapname}". Принадлежность файла ясна по факту нахождения в директории /_Ganimed/, а ID хранятся в прототипе внутри файла, так что это не повлияет на них.
Тем не менее, сами по себе эти изменения мне кажутся незначительными (просто мелкие фиксы/перестановки/добавления) одного предмета тут и там, как по-мне можно применить эти изменения на существующие поддерживаемые карты ADT. Но если ты будешь следить за всеми АДТ ремаппами и вовремя актуализировать карты (что сложно), можно сделать и так.
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Signed-off-by: Hyper B <137433177+HyperB1@users.noreply.github.com>
Описание PR
Название говорит само за себя?
Мерджить через squash
Технические детали
Медиа
Чек-лист
Список изменений
Общие изменения
Aspid
Saltern
Чейнджог не требуется, чисто технические и незначительные фиксы по большей части.