Пых имплант. #386
Пых имплант. #386
Conversation
VunderFaffle
requested review from
BL02DL,
Flybik and
Evgencheg
as code owners
|
Warning Rate limit exceeded@VunderFaffle has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 6 minutes and 42 seconds before requesting another review. ⌛ 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. 📒 Files selected for processing (2)
WalkthroughИзменения представляют собой добавление нового импланта "Weh" в игру. Это включает создание новой сущности импланта, действия для его активации, локализацию на английском и русском языках, а также добавление грузового продукта и ящика с имплантами. Новый имплант позволяет пользователю выполнять действие "Weh" с определенной задержкой и звуковым эффектом. Changes
Sequence DiagramsequenceDiagram
participant Игрок
participant Имплант
participant Действие
participant Звук
Игрок->>Имплант: Активировать
Имплант->>Действие: Запустить действие
Действие-->>Игрок: Задержка 1 сек
Действие->>Звук: Воспроизвести "Weh!"
Звук-->>Игрок: Звуковой эффект
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
Изменения: Локализация
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🔭 Outside diff range comments (1)
Resources/Prototypes/_LostParadise/Entities/Objects/Misc/implanter.yml (1)
Line range hint
1-8: Исправьте опечатку в существующем импланте!В названии и описании импланта "loalty" допущена опечатка.
- name: loalty implant - description: Do your enemies your friends! + name: loyalty implant + description: Make your enemies your friends!🧰 Tools
🪛 yamllint (1.35.1)
[error] 15-15: too many spaces inside brackets
(brackets)
[error] 15-15: too many spaces inside brackets
(brackets)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (10)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/action/types.ftl(1 hunks)Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/crates/weh_implant_crate.ftl(1 hunks)Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/objects/misc/implanter.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/action/types.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/catalog/fills/crates/weh_implant_crate.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/misc/implanter.ftl(1 hunks)Resources/Prototypes/_LostParadise/Action/types.yml(1 hunks)Resources/Prototypes/_LostParadise/Catalog/Cargo/cargo_fun.yml(1 hunks)Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/weh_implant_crate.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Objects/Misc/implanter.yml(3 hunks)
🧰 Additional context used
📓 Path-based instructions (10)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/action/types.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/action/types.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/crates/weh_implant_crate.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/crates/weh_implant_crate.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/Catalog/Cargo/cargo_fun.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/weh_implant_crate.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Action/types.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/objects/misc/implanter.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/objects/misc/implanter.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/Entities/Objects/Misc/implanter.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
🪛 yamllint (1.35.1)
Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/weh_implant_crate.yml
[error] 88-88: trailing spaces
(trailing-spaces)
[error] 95-95: wrong indentation: expected 4 but found 2
(indentation)
Resources/Prototypes/_LostParadise/Action/types.yml
[error] 17-17: too many spaces inside brackets
(brackets)
[error] 17-17: too many spaces inside brackets
(brackets)
[error] 19-19: wrong indentation: expected at least 3
(indentation)
[error] 20-20: too many spaces inside braces
(braces)
[error] 20-20: too many spaces inside braces
(braces)
Resources/Prototypes/_LostParadise/Entities/Objects/Misc/implanter.yml
[error] 15-15: too many spaces inside brackets
(brackets)
[error] 15-15: too many spaces inside brackets
(brackets)
[error] 41-41: trailing spaces
(trailing-spaces)
[error] 50-50: too many blank lines
(1 > 0) (empty-lines)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (7)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/action/types.ftl (1)
1-2:⚠️ Potential issueИсправьте регистр первой буквы перевода.
Согласно правилам локализации, все переводы, начинающиеся с 'ent-', должны начинаться с заглавной буквы после знака '='.
-ent-LPPActionActivateWehImplant = пых! +ent-LPPActionActivateWehImplant = Пых!Likely invalid or redundant comment.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/action/types.ftl (1)
1-2: Локализация корректна!Правильное использование формата FTL и корректная капитализация для сущности.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/catalog/fills/crates/weh_implant_crate.ftl (1)
1-2: Локализация ящика соответствует требованиям!Описание чётко указывает содержимое и предназначение ящика.
Resources/Prototypes/_LostParadise/Catalog/Cargo/cargo_fun.yml (1)
11-19: Конфигурация товара корректна!Цена и категоризация соответствуют требованиям PR. Использование иконки импланта уместно.
Resources/Prototypes/_LostParadise/Action/types.yml (2)
13-22: Реализация действия корректна!
- Иконка соответствует тематике (плюшевая игрушка)
- Задержка в 1 секунду предотвращает спам
- Корректная интеграция с системой имплантов
🧰 Tools
🪛 yamllint (1.35.1)
[error] 17-17: too many spaces inside brackets
(brackets)
[error] 17-17: too many spaces inside brackets
(brackets)
[error] 19-19: wrong indentation: expected at least 3
(indentation)
[error] 20-20: too many spaces inside braces
(braces)
[error] 20-20: too many spaces inside braces
(braces)
17-17: 🧹 Nitpick (assertive)Исправьте форматирование!
Уберите лишние пробелы внутри квадратных скобок.
- categories: [ HideSpawnMenu ] + categories: [HideSpawnMenu]Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 17-17: too many spaces inside brackets
(brackets)
[error] 17-17: too many spaces inside brackets
(brackets)
Resources/Prototypes/_LostParadise/Entities/Objects/Misc/implanter.yml (1)
10-31: Реализация нового импланта корректна!
- Правильное наследование от базовых классов
- Корректная интеграция со звуковой системой
- Компоненты настроены правильно
🧰 Tools
🪛 yamllint (1.35.1)
[error] 15-15: too many spaces inside brackets
(brackets)
[error] 15-15: too many spaces inside brackets
(brackets)
|
|
||
|
|
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Удалите пустые строки в начале файла.
Файл не должен начинаться с пустых строк.
...ces/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/catalog/fills/crates/weh_implant_crate.ftl
Outdated
Show resolved
Hide resolved
| ent-LPPWehImplant = Пых имплант | ||
| .desc = Позволяет пользователю ПЫХАТЬ | ||
| ent-LPPWehImplanter = Пых имплант | ||
| .desc = Пых! |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Обеспечьте единообразие в описаниях.
- Используйте одинаковый стиль написания слова "пых" во всех описаниях.
- Слово "ПЫХАТЬ" в верхнем регистре выглядит слишком агрессивно.
ent-LPPWehImplant = Пых имплант
- .desc = Позволяет пользователю ПЫХАТЬ
+ .desc = Позволяет пользователю пыхать
ent-LPPWehImplanter = Пых имплант
.desc = Пых!📝 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.
| ent-LPPWehImplant = Пых имплант | |
| .desc = Позволяет пользователю ПЫХАТЬ | |
| ent-LPPWehImplanter = Пых имплант | |
| .desc = Пых! | |
| ent-LPPWehImplant = Пых имплант | |
| .desc = Позволяет пользователю пыхать | |
| ent-LPPWehImplanter = Пых имплант | |
| .desc = Пых! |
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/objects/misc/implanter.ftl
Outdated
Show resolved
Hide resolved
| @@ -85,3 +85,14 @@ | |||
| contents: | |||
| - id: MacroBombImplanter | |||
| amount: 5 | |||
|
|
|||
There was a problem hiding this comment.
Удалите пробелы в конце строки
В строке 88 обнаружены лишние пробелы в конце строки. Это может вызвать проблемы при обработке файла.
-
+🧰 Tools
🪛 yamllint (1.35.1)
[error] 88-88: trailing spaces
(trailing-spaces)
| - type: entity | ||
| id: LPPCrateWehImplants | ||
| name: Weh implant crate | ||
| description: Crate filled with 5 weh implants which let the user weh | ||
| parent: CrateMedical | ||
| components: | ||
| - type: StorageFill | ||
| contents: | ||
| - id: LPPWehImplanter | ||
| amount: 5 |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружено несоответствие в типе родительского контейнера
Все остальные имплантаты в файле используют CrateSecgear в качестве родительского типа. Рекомендуется изменить тип контейнера для Weh-имплантата с CrateMedical на CrateSecgear для соблюдения единообразия.
🔗 Analysis chain
Проверьте корректность использования CrateMedical
Структура нового импланта выглядит правильно, но в отличие от других имплантов в файле, которые используют CrateSecgear, данный имплант использует CrateMedical. Пожалуйста, подтвердите, что это намеренное изменение.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверяем использование CrateMedical в других имплантах
rg "parent: CrateMedical" Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/
# Проверяем определение базового типа
rg "id: CrateMedical" Resources/Prototypes/
Length of output: 729
Script:
#!/bin/bash
# Check for CrateSecgear usage in implant crates and its definition
rg "parent: CrateSecgear" Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/
rg "id: CrateSecgear" Resources/Prototypes/
Length of output: 1024
Script:
#!/bin/bash
# Check the full content of the weh implant crate file
cat Resources/Prototypes/_LostParadise/Catalog/Fills/Crates/weh_implant_crate.yml
Length of output: 2864
🧰 Tools
🪛 yamllint (1.35.1)
[error] 95-95: wrong indentation: expected 4 but found 2
(indentation)
…g/fills/crates/weh_implant_crate.ftl Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…es/objects/misc/implanter.ftl Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
Хз почему билд и тест дебаг не прошли. Не во мне причина, у меня всё протестировано. Там буквально нечему ломаться. |
Evgencheg
merged commit 7587db7
into
Lost-Paradise-Project:master
Описание обновления:
Пых имплант, позволяющий пыхать.
При активации импланта издаётся звук "weh"
Импланты можно приобрести в карго.
Ящик 5 имплантов - 2К кредитов.
Медиа
Проверки
Изменения