[Feature] Flags #151
[Feature] Flags #151
Conversation
WalkthroughВ данном запросе на изменение добавлены новые записи в файл локализации для русского языка, касающиеся флагов различных сущностей и регионов. В файле Changes
Suggested labels
Suggested reviewers
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 (
|
|
RSI Diff Bot; head commit b144b18 merging into 74256db Resources/Textures/_White/Structures/Wallmounts/flags.rsi
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Outside diff range and nitpick comments (2)
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl (1)
5-6: Проверьте уместность неформального тона.Некоторые описания используют неформальный тон ("Круто!!!", "Великолепно"), что может не соответствовать общему стилю игры.
Also applies to: 9-10
Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml (1)
44-45: Рекомендуется пересмотреть описания политически чувствительного контентаНекоторые описания флагов содержат потенциально чувствительные политические и исторические отсылки. Рекомендуется:
- Использовать более нейтральные описания
- Избегать упоминаний реальных исторических событий
- Сфокусироваться на внутриигровом лоре
Also applies to: 154-155, 192-194, 222-223
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (25)
Resources/Textures/_White/Structures/Wallmounts/flags.rsi/agurk.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/belarus.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/california.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/california_solfed.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/chechnya.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/coder.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/germany.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/germany_solfed.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/goldring.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/hca.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/inteq.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/lizard.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/mothic.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/nri.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/nt.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/russian.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/shigu.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/solfed.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/solfed_alt.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/soviet.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/syndicate.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/tizira.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/ukraine.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/usa.pngis excluded by!**/*.pngResources/Textures/_White/Structures/Wallmounts/flags.rsi/vulp.pngis excluded by!**/*.png
📒 Files selected for processing (5)
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl(1 hunks)Resources/Migrations/migration.yml(1 hunks)Resources/Prototypes/Entities/Markers/Spawners/Random/posters.yml(2 hunks)Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml(1 hunks)Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json
🔇 Additional comments (3)
Resources/Prototypes/Entities/Markers/Spawners/Random/posters.yml (2)
179-205: Проверьте интеграцию легальных флагов.
Большое количество легальных флагов добавлено в систему. Убедитесь, что:
- Все прототипы флагов существуют
- Флаги правильно категоризированы как legit
✅ Verification successful
Все прототипы флагов найдены и корректно определены
Проверка показала, что:
- Все флаги из списка имеют соответствующие прототипы в
Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml - Каждый флаг имеет корректное определение с именем и описанием
- Все прототипы правильно определены как настенные декоративные элементы
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify all legit flag prototypes exist
for flag in $(rg -o 'Flag\w+' Resources/Prototypes/Entities/Markers/Spawners/Random/posters.yml | sort -u); do
rg -l "^- type: entity.*id: $flag" "Resources/Prototypes"
done
Length of output: 4423
Script:
#!/bin/bash
# Let's search for flag prototypes in a more specific location and with broader pattern
rg -l "id: Flag" Resources/Prototypes/Entities/Objects/Decoration/flags.yml
# Also let's check the content to verify legit categorization
rg "parent: BasePoster|id: Flag" Resources/Prototypes/Entities/Objects/Decoration/flags.yml
Length of output: 369
Script:
#!/bin/bash
# Поиск файлов с флагами в любом месте репозитория
fd flags.yml
# Поиск всех прототипов флагов в любом месте
rg "id: Flag\w+" Resources/Prototypes -A 2
Length of output: 10341
99-106: Проверьте корректность распределения вероятностей.
Изменение вероятности с 1.0 на 0.99 и добавление редких прототипов с вероятностью 0.01 выглядит корректным, но рекомендуется проверить:
- Сумма вероятностей равна 1.0
- Редкие прототипы правильно категоризированы как contraband
✅ Verification successful
Распределение вероятностей и категоризация корректны
Проверка показала:
- Сумма вероятностей (0.99 + 0.01) равна 1.0
- Все три редких флага (FlagCoder, FlagSyndicate, FlagInteQ) существуют в кодовой базе
- Флаги правильно категоризированы как contraband, судя по их описания:
- FlagSyndicate - флаг повстанческой организации
- FlagInteQ - флаг ЧВК со "следами крови"
- FlagCoder - нестандартный флаг
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify flag prototypes exist and are properly categorized
for flag in FlagCoder FlagSyndicate FlagInteQ; do
rg -l "^- type: entity.*id: $flag" "Resources/Prototypes"
done
Length of output: 333
Script:
#!/bin/bash
# Let's try a broader search for flag entities and their definitions
rg -l "flag" "Resources/Prototypes" --type yaml
# Then let's look for specific contraband or rare flags
rg "contraband|rare.*flag" "Resources/Prototypes" -C 5 --type yaml
Length of output: 63789
Script:
#!/bin/bash
# Let's check for flag entity definitions in the codebase
rg -l "FlagCoder|FlagSyndicate|FlagInteQ" "Resources/Prototypes" --type yaml
# And let's look at their content if they exist
rg "id: (FlagCoder|FlagSyndicate|FlagInteQ)" "Resources/Prototypes" -B 5 -A 10 --type yaml
Length of output: 4947
Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml (1)
8-9: Необходима проверка наличия спрайтов
Все определения флагов используют спрайты из файла _White/Structures/Wallmounts/flags.rsi. Необходимо убедиться в наличии всех состояний спрайтов.
Also applies to: 18-19, 28-29, 38-39, 48-49, 58-59, 68-69, 78-79, 88-89, 98-99, 108-109, 118-119, 128-129, 138-139, 148-149, 158-159, 168-169, 178-179, 186-187, 196-197, 206-207, 216-217, 226-227, 236-237, 246-247
✅ Verification successful
Все спрайты присутствуют и корректно определены
Сверка показала полное соответствие между состояниями спрайтов, используемыми в flags.yml (25 состояний), и состояниями, определёнными в meta.json файле спрайтов (25 состояний). Все необходимые спрайты присутствуют в ресурсах.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия файла спрайтов и всех необходимых состояний
# Проверяем наличие файла .rsi и его метаданных
if [ -f "Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json" ]; then
echo "Файл flags.rsi найден"
# Извлекаем список состояний из meta.json
jq '.states | keys[]' "Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json"
else
echo "ОШИБКА: Файл flags.rsi не найден"
fi
Length of output: 290
Script:
#!/bin/bash
# Получаем список всех используемых состояний спрайтов из yml файла
echo "Состояния спрайтов, используемые в flags.yml:"
rg "state: " "Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml" | awk '{print $2}'
echo -e "\nСостояния спрайтов из meta.json:"
jq -r '.states | to_entries[] | .value.name // empty' "Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json"
Length of output: 796
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl
Outdated
Show resolved
Hide resolved
Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml
Outdated
Show resolved
Hide resolved
|
На счет флагов стран спорно, если оникс даст добро, замержу |
Главное что-бы не заметил отсутствие флага Эстонии |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (1)
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl (1)
3-4: Рекомендуется использовать более формальный языкТекущее описание флага кодеров использует сленговое слово "задротов". Предлагаю использовать более профессиональную формулировку.
- .desc = Флаг клуба задротов. Великолепно. + .desc = Флаг клуба программистов. Великолепно.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (4)
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl(1 hunks)Resources/Prototypes/Entities/Markers/Spawners/Random/posters.yml(2 hunks)Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml(1 hunks)Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- Resources/Textures/_White/Structures/Wallmounts/flags.rsi/meta.json
- Resources/Prototypes/Entities/Markers/Spawners/Random/posters.yml
🧰 Additional context used
📓 Learnings (1)
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl (1)
Learnt from: PuroSlavKing
PR: WWhiteDreamProject/wwdpublic#151
File: Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl:31-32
Timestamp: 2024-12-11T16:36:33.979Z
Learning: В будущих обзорах, при указании на проблемы или предлагая изменения, необходимо предоставлять конкретные предложения по улучшению.
🔇 Additional comments (2)
Resources/Prototypes/_White/Entities/Structures/Wallmounts/flags.yml (2)
91-98: Отсутствуют обязательные поля в сущности
У сущности FlagSolfedAlt отсутствуют поля name и description, которые присутствуют у других флагов.
1-127: Структура сущностей реализована корректно
Все сущности флагов имеют правильную структуру наследования и компонентов.
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl
Show resolved
Hide resolved
Resources/Locale/ru-RU/_white/prototypes/entities/structures/wallmounts/flags.ftl
Show resolved
Hide resolved
* [Feature] Flags * neutral desc * remove Ukraine and other
Описание PR
Смешные флажки. Да, это фича, что некоторые из них на своих языках.
Шанс спавна 0.01% вместо постера.
Медиа
Список
(Уже без Украины, ну и остальных стран)
Изменения