MindShield атмосу и ведущим должностям #271
Conversation
WalkthroughВ этом запросе на вытягивание внесены изменения в определения нескольких должностей, добавляющие новую секцию Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant JobDefinition
participant MindShieldImplant
User->>JobDefinition: Запрос на использование роли
JobDefinition->>MindShieldImplant: Проверка доступности импланта
MindShieldImplant-->>JobDefinition: Имплант доступен
JobDefinition-->>User: Подтверждение роли с имплантом
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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (4)
- Resources/Prototypes/Roles/Jobs/Engineering/atmospheric_technician.yml (1 hunks)
- Resources/Prototypes/Roles/Jobs/Engineering/senior_engineer.yml (1 hunks)
- Resources/Prototypes/Roles/Jobs/Medical/senior_physician.yml (1 hunks)
- Resources/Prototypes/Roles/Jobs/Science/senior_researcher.yml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
Resources/Prototypes/Roles/Jobs/Engineering/atmospheric_technician.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать изменения по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/Roles/Jobs/Engineering/senior_engineer.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать изменения по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/Roles/Jobs/Medical/senior_physician.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать изменения по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/Roles/Jobs/Science/senior_researcher.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать изменения по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🪛 yamllint
Resources/Prototypes/Roles/Jobs/Engineering/atmospheric_technician.yml
[error] 20-20: wrong indentation: expected 4 but found 2
(indentation)
[error] 21-21: too many spaces inside brackets
(brackets)
[error] 21-21: too many spaces inside brackets
(brackets)
Resources/Prototypes/Roles/Jobs/Engineering/senior_engineer.yml
[error] 26-26: wrong indentation: expected 4 but found 2
(indentation)
[error] 27-27: too many spaces inside brackets
(brackets)
[error] 27-27: too many spaces inside brackets
(brackets)
Resources/Prototypes/Roles/Jobs/Medical/senior_physician.yml
[error] 25-25: wrong indentation: expected 4 but found 2
(indentation)
[error] 26-26: too many spaces inside brackets
(brackets)
[error] 26-26: too many spaces inside brackets
(brackets)
Resources/Prototypes/Roles/Jobs/Science/senior_researcher.yml
[error] 19-19: wrong indentation: expected 4 but found 2
(indentation)
[error] 20-20: too many spaces inside brackets
(brackets)
[error] 20-20: too many spaces inside brackets
(brackets)
🔇 Additional comments (5)
Resources/Prototypes/Roles/Jobs/Science/senior_researcher.yml (2)
Line range hint
7-7: Изменение отдела подтверждено.Изменение отдела с Science на Epistemics соответствует целям PR и согласуется с комментарием DeltaV. Это изменение корректно отражает новую структуру отделов.
🧰 Tools
🪛 yamllint
[error] 15-15: wrong indentation: expected 4 but found 2
(indentation)
[warning] 16-16: comment not indented like content
(comments-indentation)
[error] 19-19: wrong indentation: expected 4 but found 2
(indentation)
[error] 20-20: too many spaces inside brackets
(brackets)
[error] 20-20: too many spaces inside brackets
(brackets)
18-20:⚠️ Potential issueДобавление MindShield подтверждено, но требуется исправление форматирования.
Добавление MindShield имплантата соответствует целям PR. Однако есть проблемы с форматированием:
- Отступ должен быть 4 пробела вместо 2.
- Лишние пробелы внутри квадратных скобок.
Пожалуйста, примените следующие изменения для исправления форматирования:
special: - - !type:AddImplantSpecial - implants: [ MindShieldImplant ] + - !type:AddImplantSpecial + implants: [MindShieldImplant]Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint
[error] 19-19: wrong indentation: expected 4 but found 2
(indentation)
[error] 20-20: too many spaces inside brackets
(brackets)
[error] 20-20: too many spaces inside brackets
(brackets)
Resources/Prototypes/Roles/Jobs/Engineering/senior_engineer.yml (2)
25-27: Одобрено добавление секцииspecialсMindShieldImplant.Добавление
MindShieldImplantдля старшего инженера соответствует целям PR и корректно реализовано в структуре YAML.🧰 Tools
🪛 yamllint
[error] 26-26: wrong indentation: expected 4 but found 2
(indentation)
[error] 27-27: too many spaces inside brackets
(brackets)
[error] 27-27: too many spaces inside brackets
(brackets)
25-27: Изменения соответствуют целям PR и руководствам по кодированию.Подтверждаю, что:
- Добавление
MindShieldImplantсоответствует целям PR.- В прототипе отсутствуют переводы на русский язык.
- Прототип находится в правильной папке.
🧰 Tools
🪛 yamllint
[error] 26-26: wrong indentation: expected 4 but found 2
(indentation)
[error] 27-27: too many spaces inside brackets
(brackets)
[error] 27-27: too many spaces inside brackets
(brackets)
Resources/Prototypes/Roles/Jobs/Medical/senior_physician.yml (1)
25-26: Одобрено добавление MindShieldImplant для SeniorPhysician.Добавление MindShieldImplant соответствует целям PR и улучшает геймплей для роли старшего врача.
🧰 Tools
🪛 yamllint
[error] 25-25: wrong indentation: expected 4 but found 2
(indentation)
[error] 26-26: too many spaces inside brackets
(brackets)
[error] 26-26: too many spaces inside brackets
(brackets)
| special: | ||
| - !type:AddImplantSpecial | ||
| implants: [ MindShieldImplant ] |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Одобрено добавление MindShield импланта, но требуется исправление форматирования.
Добавление MindShield импланта для Атмосферного техника соответствует целям PR. Однако есть проблемы с форматированием:
- Отступ для строки 20 должен быть 4 пробела вместо 2.
- В строке 21 есть лишние пробелы внутри квадратных скобок.
Предлагаю внести следующие изменения для исправления форматирования:
special:
- - !type:AddImplantSpecial
- implants: [ MindShieldImplant ]
+ - !type:AddImplantSpecial
+ implants: [MindShieldImplant]📝 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.
| special: | |
| - !type:AddImplantSpecial | |
| implants: [ MindShieldImplant ] | |
| special: | |
| - !type:AddImplantSpecial | |
| implants: [MindShieldImplant] |
🧰 Tools
🪛 yamllint
[error] 20-20: wrong indentation: expected 4 but found 2
(indentation)
[error] 21-21: too many spaces inside brackets
(brackets)
[error] 21-21: too many spaces inside brackets
(brackets)
| - !type:AddImplantSpecial | ||
| implants: [ MindShieldImplant ] |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Исправьте отступы и пробелы для улучшения читаемости.
Обнаружены следующие проблемы форматирования:
- Неправильный отступ в строке 26 (должно быть 4 пробела вместо 2).
- Лишние пробелы внутри квадратных скобок в строке 27.
Предлагаю внести следующие изменения:
- special:
- - !type:AddImplantSpecial
- implants: [ MindShieldImplant ]
+ special:
+ - !type:AddImplantSpecial
+ implants: [MindShieldImplant]Committable suggestion was skipped due to low confidence.
🧰 Tools
🪛 yamllint
[error] 26-26: wrong indentation: expected 4 but found 2
(indentation)
[error] 27-27: too many spaces inside brackets
(brackets)
[error] 27-27: too many spaces inside brackets
(brackets)
| - !type:AddImplantSpecial | ||
| implants: [ MindShieldImplant ] |
There was a problem hiding this comment.
Исправьте отступы и пробелы в новых строках.
Обнаружены проблемы с форматированием:
- Неправильный отступ в строке 25. Ожидается 4 пробела, но найдено 2.
- Лишние пробелы внутри квадратных скобок в строке 26.
Примените следующие изменения для исправления форматирования:
- - !type:AddImplantSpecial
- implants: [ MindShieldImplant ]
+ - !type:AddImplantSpecial
+ implants: [MindShieldImplant]📝 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.
| - !type:AddImplantSpecial | |
| implants: [ MindShieldImplant ] | |
| - !type:AddImplantSpecial | |
| implants: [MindShieldImplant] |
🧰 Tools
🪛 yamllint
[error] 25-25: wrong indentation: expected 4 but found 2
(indentation)
[error] 26-26: too many spaces inside brackets
(brackets)
[error] 26-26: too many spaces inside brackets
(brackets)
Описание обновления:
Добавлен Щит Разума Атмосферным Техникам, Ведущим Инженерам, Ведущим Врачам и Ведущим Учёным.
Медиа
Проверки
Изменения
🆑 Farrellka