Armbands 1 #304
Armbands 1 #304
Conversation
ПроцессВ данном пулл-запросе внесены изменения, касающиеся локализации и конфигурации сущностей в проекте. Добавлены новые записи в файлы локализации для английского и русского языков, включая описание различных цветных нарукавных повязок и коробки для нарукавных повязок. Также были обновлены YAML-файлы, определяющие новые сущности, такие как нарукавные повязки и различные типы латр, а также добавлены новые рецепты для их создания. Кроме того, созданы метаданные для текстур нарукавных повязок и коробок, что улучшает визуальное представление этих объектов в игре. Изменения
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
Статус: Требуется одобрение
|
RSI Diff Bot; head commit 1f4b16b merging into 91cfd1c Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi
Resources/Textures/_LostParadise/Objects/Storage/boxes.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: 18
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
⛔ Files ignored due to path filters (23)
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/equipped-EYES-tajaran.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/equipped-EYES-vulpkanin.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/equipped-EYES.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Storage/boxes.rsi/armband.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Storage/boxes.rsi/box_security.pngis excluded by!**/*.png
📒 Files selected for processing (17)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1 hunks)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
- Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (2 hunks)
- Resources/Prototypes/Entities/Structures/Machines/lathe.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Catalog/Fills/Boxes/security.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Recipes/Lathes/ClothinglatheRecipe.yml (1 hunks)
- Resources/Prototypes/_LostParadise/tags.yml (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi/meta.json (1 hunks)
- Resources/Textures/_LostParadise/Objects/Storage/boxes.rsi/meta.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (10)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/Entities/Structures/Machines/lathe.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Catalog/Fills/Boxes/security.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Recipes/Lathes/ClothinglatheRecipe.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/tags.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🪛 yamllint
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml
[warning] 339-339: missing starting space in comment
(comments)
[warning] 339-339: comment not indented like content
(comments-indentation)
[warning] 370-370: missing starting space in comment
(comments)
[warning] 370-370: comment not indented like content
(comments-indentation)
Resources/Prototypes/_LostParadise/Catalog/Fills/Boxes/security.yml
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 9-9: wrong indentation: expected at least 5
(indentation)
[error] 12-12: wrong indentation: expected at least 5
(indentation)
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml
[error] 6-6: wrong indentation: expected at least 3
(indentation)
[error] 9-9: wrong indentation: expected at least 5
(indentation)
[error] 18-18: wrong indentation: expected 10 but found 8
(indentation)
[error] 22-22: wrong indentation: expected 6 but found 4
(indentation)
[error] 30-30: wrong indentation: expected 6 but found 4
(indentation)
[error] 37-37: trailing spaces
(trailing-spaces)
[error] 39-39: wrong indentation: expected 4 but found 2
(indentation)
[error] 45-45: wrong indentation: expected 6 but found 4
(indentation)
[error] 53-53: wrong indentation: expected 4 but found 2
(indentation)
[error] 59-59: wrong indentation: expected 6 but found 4
(indentation)
[error] 67-67: wrong indentation: expected 4 but found 2
(indentation)
[error] 73-73: wrong indentation: expected 6 but found 4
(indentation)
[error] 81-81: wrong indentation: expected 4 but found 2
(indentation)
[error] 87-87: wrong indentation: expected 6 but found 4
(indentation)
[error] 95-95: wrong indentation: expected 4 but found 2
(indentation)
[error] 101-101: wrong indentation: expected 6 but found 4
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 115-115: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/_LostParadise/Recipes/Lathes/ClothinglatheRecipe.yml
[error] 759-759: no new line character at the end of file
(new-line-at-end-of-file)
[error] 759-759: trailing spaces
(trailing-spaces)
🔇 Additional comments (26)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
1-2: Переводы соответствуют требованиям!Корректное использование заглавных букв после знака равенства для ent- записей. Перевод логичный и соответствует контексту игры.
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/meta.json (2)
1-4: Метаданные корректны!Лицензия и информация об авторских правах указаны правильно.
5-8: Размеры текстуры соответствуют стандартам!Размер 32x32 пикселя является стандартным для спрайтов игры.
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_green.rsi/meta.json (3)
5-8: Размер текстуры соответствует стандартам.Размер 32x32 является стандартным для спрайтов в проекте.
1-4: 🧹 Nitpick (assertive)Проверьте правильность лицензии и авторских прав.
Убедитесь, что:
- Лицензия CC-BY-SA-3.0 соответствует политике проекта Lost Paradise
- Все авторы (
nelepus & miller4040) дали согласие на использование их работыРекомендуется добавить год создания в информацию об авторских правах:
- "copyright": "Made by nelepus & miller4040", + "copyright": "2024, Made by nelepus & miller4040",
9-21: 🧹 Nitpick (assertive)Проверьте состояния экипировки.
В соответствии с целями PR, повязка должна надеваться в слот шеи, но состояние
equipped-HELMETможет указывать на неиспользуемую функциональность.Рекомендуется:
- Документировать причину наличия состояния
equipped-HELMET, если оно планируется к использованию в будущем- Удалить неиспользуемое состояние, если оно не планируется к применению
✅ Verification successful
Based on the search results, I can see that
equipped-HELMETis a commonly used state for head slot equipment across many items in the game, including armbands. This appears to be a standard equipment state alongsideequipped-NECKfor items that can be worn on either the head or neck slots.Let me generate the final response in Russian:
Состояние
equipped-HELMETявляется допустимым для повязок.Анализ кодовой базы показывает, что состояние
equipped-HELMETшироко используется для предметов, которые могут надеваться как на шею, так и на голову, включая другие повязки в директорииResources/Textures/_LostParadise/Clothing/Uniforms/Armband/. Это стандартная практика в кодовой базе.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Проверка использования состояния HELMET в других файлах проекта # Поиск упоминаний HELMET в связанных файлах rg -g "*.yml" -g "*.json" "equipped-HELMET"Length of output: 118758
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/meta.json (1)
1-8: Метаданные корректно настроены!Файл содержит всю необходимую информацию о версии, лицензии и авторских правах.
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/meta.json (1)
5-8: Корректная конфигурация размера спрайта.Размер 32x32 пикселя соответствует стандартным требованиям игровых спрайтов.
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_yellow.rsi/meta.json (2)
1-8: Метаданные корректны!Все необходимые поля присутствуют и правильно заполнены, включая лицензию и информацию об авторских правах.
9-22: Рассмотрите добавление состояний для наложения на униформу.Текущие состояния корректны для ношения в слоте шеи и шлема. Однако, учитывая планы по реализации ношения повязок поверх униформы, рекомендуется предусмотреть дополнительные состояния для наложения (overlay).
Проверим существующие состояния в других файлах повязок:
Resources/Textures/_LostParadise/Objects/Storage/boxes.rsi/meta.json (1)
4-4: Проверьте существование указанного коммитаНеобходимо убедиться, что указанный коммит существует в репозитории tgstation.
✅ Verification successful
Коммит успешно подтвержден
Указанный коммит
cc65477c04f7403ca8a457bd5bae69a01abadbf0существует в репозитории tgstation/tgstation.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the referenced commit exists in the tgstation repository COMMIT="cc65477c04f7403ca8a457bd5bae69a01abadbf0" # Use GitHub API to check if the commit exists gh api \ -H "Accept: application/vnd.github+json" \ repos/tgstation/tgstation/commits/$COMMIT \ --jq '.sha'Length of output: 225
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_white.rsi/meta.json (2)
1-4: Метаданные корректно определены!Лицензия и информация об авторских правах указаны правильно.
5-8: Размер текстуры соответствует стандартам!Указанный размер 32x32 пикселя является стандартным для спрайтов игры.
Resources/Prototypes/_LostParadise/Catalog/Fills/Boxes/security.yml (4)
1-5: Базовая конфигурация сущности корректна!Определение сущности соответствует требованиям и находится в правильной директории.
16-29: Проверить существование всех типов нарукавных повязокНеобходимо убедиться, что все указанные ID нарукавных повязок определены в системе.
#!/bin/bash # Check if all armband entities are defined for id in LPPArmbandRed LPPArmbandGreen LPPArmbandBlue LPPArmbandWhite LPPArmbandYellow LPPArmbandPurple; do echo "Checking $id:" rg "id: $id" "Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml" done
30-34: Проверить наличие спрайтовНеобходимо убедиться, что файл спрайтов и указанные состояния существуют.
#!/bin/bash # Check if the sprite file exists fd "boxes.rsi" "Resources/Textures/_LostParadise/Objects/Storage/" # Check if states are defined in the metadata if [ -f "Resources/Textures/_LostParadise/Objects/Storage/boxes.rsi/meta.json" ]; then echo "Checking states in metadata:" jq '.states | keys' "Resources/Textures/_LostParadise/Objects/Storage/boxes.rsi/meta.json" fi
10-15: Проверить существование тега LPPArmbandНеобходимо убедиться, что тег определен в системе.
✅ Verification successful
Let me gather more information to verify the tag usage.
Let me check the content of the tags.yml file to verify the tag definition.
Тег LPPArmband корректно определен в системе
Тег
LPPArmbandнайден в файлеResources/Prototypes/_LostParadise/tags.ymlи правильно определен как "Нарукавная повязка". Он также используется в нескольких других файлах системы, включая рецепты и прототипы сущностей, что подтверждает его активное использование.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Check if the tag is defined rg "LPPArmband" "Resources/Prototypes/**/tags.yml"Length of output: 377
Script:
#!/bin/bash # Search for LPPArmband tag across all files rg -l "LPPArmband" # Also check for any yml files that might define tags fd -e yml -e yaml | xargs rg "LPPArmband" -lLength of output: 897
Script:
#!/bin/bash # Check the content of tags.yml to verify tag definition cat "Resources/Prototypes/_LostParadise/tags.yml"Length of output: 2485
🧰 Tools
🪛 yamllint
[error] 12-12: wrong indentation: expected at least 5
(indentation)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
1-2: Перевод корректен!Правильное использование заглавной буквы после "=" для ent- идентификатора. Уникальное описание для красной повязки.
Resources/Prototypes/_LostParadise/tags.yml (1)
172-173:⚠️ Potential issueНеобходимо заменить русский комментарий на английский.
Тег добавлен корректно, но согласно правилам кодирования, в прототипах запрещены комментарии на русском языке.
Примените следующие изменения:
- id: LPPArmband # Нарукавная повязка + id: LPPArmband # ArmbandLikely invalid or redundant comment.
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml (1)
1-31: 🧹 Nitpick (assertive)Необходимо добавить документацию и валидацию
Обнаружены следующие проблемы в базовом прототипе:
- Отсутствует документация о назначении компонента
Food- Отсутствует валидация для
maxVol: 5- Комментарий на строке 3 указывает на временное решение
Предлагаю:
- Добавить комментарии, объясняющие необходимость компонента
Food- Создать задачу для отслеживания будущих изменений реализации
🧰 Tools
🪛 yamllint
[error] 6-6: wrong indentation: expected at least 3
(indentation)
[error] 9-9: wrong indentation: expected at least 5
(indentation)
[error] 18-18: wrong indentation: expected 10 but found 8
(indentation)
[error] 22-22: wrong indentation: expected 6 but found 4
(indentation)
[error] 30-30: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)
340-340: Добавление коробки с повязками одобреноРазмещение
LPPBoxArmbandв шкафчиках главы службы безопасности логично соответствует целям PR по внедрению повязок для идентификации сотрудников службы безопасности.Also applies to: 371-371
Resources/Prototypes/_LostParadise/Recipes/Lathes/ClothinglatheRecipe.yml (4)
716-717: Корректное добавление секции для нарукавных повязок!Заголовок секции правильно отформатирован и соответствует стандартам YAML.
718-759: Рецепты корректно структурированы!Все рецепты имеют одинаковую структуру и требования к материалам, что обеспечивает консистентность. Время создания в 1 единицу и требование 100 единиц ткани выглядят разумно для небольших предметов одежды.
🧰 Tools
🪛 yamllint
[error] 759-759: no new line character at the end of file
(new-line-at-end-of-file)
[error] 759-759: trailing spaces
(trailing-spaces)
759-759:⚠️ Potential issueНеобходимо исправить форматирование файла.
Обнаружены следующие проблемы форматирования:
- Отсутствует пустая строка в конце файла
- Присутствуют пробелы в конце строки 759
Примените следующие изменения:
- +Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint
[error] 759-759: no new line character at the end of file
(new-line-at-end-of-file)
[error] 759-759: trailing spaces
(trailing-spaces)
718-759: Проверьте интеграцию с принтером униформы.Согласно целям PR, эти рецепты должны быть доступны в принтере униформы. Необходимо убедиться, что они правильно интегрированы.
✅ Verification successful
Let me check the content of these files to verify the integration.
Повязки правильно интегрированы в принтер униформы
Проверка показала, что:
- Все повязки (LPPArmbandRed/Green/Blue/White/Yellow/Purple) добавлены в список рецептов принтера униформы
- Сущности повязок корректно определены с необходимыми компонентами
- Рецепты для печати повязок правильно настроены с требуемыми материалами (100 ткани) и временем изготовления
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверяем, что рецепты правильно подключены к принтеру униформы rg -l "LPPArmband(Red|Green|Blue|White|Yellow|Purple)" --type yamlLength of output: 345
Script:
#!/bin/bash # Проверяем содержимое файлов для анализа интеграции echo "=== Checking uniform printer configuration ===" rg "LPPArmband(Red|Green|Blue|White|Yellow|Purple)" Resources/Prototypes/Entities/Structures/Machines/lathe.yml -C 5 echo -e "\n=== Checking armband entities ===" cat Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml echo -e "\n=== Checking lathe recipes ===" cat Resources/Prototypes/_LostParadise/Recipes/Lathes/ClothinglatheRecipe.ymlLength of output: 17079
🧰 Tools
🪛 yamllint
[error] 759-759: no new line character at the end of file
(new-line-at-end-of-file)
[error] 759-759: trailing spaces
(trailing-spaces)
Resources/Prototypes/Entities/Structures/Machines/lathe.yml (1)
1207-1213: Корректное добавление рецептов нарукавных повязок!Рецепты корректно добавлены в секцию
staticRecipesпринтера униформы.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl
Show resolved
Hide resolved
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_blue.rsi/meta.json
Show resolved
Hide resolved
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_purple.rsi/meta.json
Show resolved
Hide resolved
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/meta.json
Show resolved
Hide resolved
Resources/Textures/_LostParadise/Clothing/Uniforms/Armband/armband_red.rsi/meta.json
Show resolved
Hide resolved
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Outdated
Show resolved
Hide resolved
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Outdated
Show resolved
Hide resolved
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/armbands.yml
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🔇 Additional comments (2)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
1-12:⚠️ Potential issueНеобходимо использовать ссылки для повторяющихся переводов
В описаниях повязок содержится повторяющийся текст "reflecting affiliation with a specific patrol group". Согласно правилам локализации, нужно использовать систему ссылок.
Примените следующие изменения:
ent-LPPArmbandRed = Red armband .desc = A red armband. Smells like blood. I guess. ent-LPPArmbandGreen = Green armband - .desc = Green armband, reflecting affiliation with a specific patrol group. + .desc = Green armband, { patrol-group-desc } ent-LPPArmbandBlue = Blue armband .desc = Blue armband. The color of this armband is said to make some officers twitch. ent-LPPArmbandYellow = Yellow armband - .desc = Yellow armband, reflecting affiliation with a certain patrol group. + .desc = Yellow armband, { patrol-group-desc } ent-LPPArmbandPurple = Magenta armband - .desc = Magenta armband, reflecting affiliation with a specific patrol group. Wearing this armband in combination with the red uniform is officially recognized as a violation of Article 106 of the current Corporation Law. + .desc = Magenta armband, { patrol-group-desc }. Wearing this armband in combination with the red uniform is officially recognized as a violation of Article 106 of the current Corporation Law. ent-LPPArmbandWhite = White armband - .desc = White armband, reflecting affiliation with a specific patrol group. + .desc = White armband, { patrol-group-desc } +patrol-group-desc = reflecting affiliation with a specific patrol groupResources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
1-2:⚠️ Potential issueИсправьте регистр первой буквы после знака равенства
Согласно правилам кодирования, все записи, начинающиеся с 'ent-', должны иметь заглавную букву после знака равенства.
-ent-LPPArmbandRed = красная нарукавная повязка +ent-LPPArmbandRed = Красная нарукавная повязкаLikely invalid or redundant comment.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Outdated
Show resolved
Hide resolved
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Show resolved
Hide resolved
|
Так, у меня сломался ПР после мерджа мастера с ориджина, я перезалью его |
|
Отмена, я дурачок, после мерджа просто забыл build прожать, ПР работает, проверил прямо сейчас |
…es/clothing/uniforms/armbands.ftl Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…g/fills/boxes/security.ftl Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1 hunks)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🔇 Additional comments (1)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
1-2: Перевод соответствует требованиям!Локализация корректна и соответствует правилам оформления:
- Правильное использование заглавной буквы для ent- префикса
- Структура перевода соответствует формату .ftl
- Содержание соответствует функционалу коробки с нарукавными повязками
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Outdated
Show resolved
Hide resolved
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1 hunks)
- Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🔇 Additional comments (1)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/boxes/security.ftl (1)
1-2: Предыдущий комментарий всё ещё актуаленНеобходимо исправить регистр текста после знака
=согласно правилам локализации для ключей, начинающихся сent-.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Show resolved
Hide resolved
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Show resolved
Hide resolved
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/armbands.ftl
Show resolved
Hide resolved
Мы добавили нарукавные повязки для визуальной идентификации отрядов СБ. Их можно получить в коробке в шкафу ГСБ или распечатать на принтере униформы ГП.
На текущий момент они надеваются в слот шеи как нарукавные повязки, или как повязки на лоб, если кому-то вдруг придет в голову засунуть их в слот шлема. В будущем планируется рассмотрение возможности надевать их поверх униформы.
Медиа
Повязки на различной форме СБ:
Версия на лбу:
Проверки
Изменения
🆑 NeLepus