Skip to content

Респрайты от пепера#1057

Closed
RJD1TOP1 wants to merge 1 commit into
space-sunrise:masterfrom
RJD1TOP1:geger
Closed

Респрайты от пепера#1057
RJD1TOP1 wants to merge 1 commit into
space-sunrise:masterfrom
RJD1TOP1:geger

Conversation

@RJD1TOP1

Copy link
Copy Markdown
Contributor

Очень красиво

Краткое описание | Short description

Рп замена спрайтов от пепера

Очень красиво
@RJD1TOP1 RJD1TOP1 requested a review from WardexOfficial as a code owner June 14, 2026 13:33
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (21)
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/equipped-BACKPACK.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/equipped.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/icon.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/inhand-left.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/inhand-right.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Backpacks/security.rsi/open b.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/equipped-BACKPACK.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/equipped.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/icon.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/inhand-left.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/inhand-right.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Duffels/security.rsi/open.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/closed.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/equipped-BACKPACK.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/equipped.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/icon.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/inhand-left.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/inhand-right.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/left hand.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/open.png is excluded by !**/*.png
  • Resources/Textures/Clothing/Back/Satchels/security.rsi/right hand.png is excluded by !**/*.png

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: a925ef34-0967-4232-8a45-e69a0e73ca0b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@RJD1TOP1

Copy link
Copy Markdown
Contributor Author

@CodeRabbit покрой меня 3 сортным говном за этот нищий ПР

@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
gh pr view 1057 --repo space-sunrise/project-fire --json files,additions,deletions,changedFiles | jq .
gh pr diff 1057 --repo space-sunrise/project-fire --name-only

Length of output: 5381



@RJD1TOP1: Ладно, держись — ты сам попросил 🐇


Официальный разбор полётов от пушистого ревьюера:

1. 🚨 Пробелы в именах файлов — серьёзно??

  • open b.png — что такое "b"? Открытый "б"? Это не имя файла, это опечатка.
  • left hand.png, right hand.png — пробелы в именах файлов в RSI это билет в однострочный баг-репорт. Стандарт проекта: inhand-left.png / inhand-right.png. Уже было в остальных папках, но для сэтчела решили поизобретать велосипед.

2. 💀 equipped-BACKPACK.pngequipped.png везде
Удалил старый стейт, добавил новый с другим именем — а YAML-определения сущностей обновил? Спойлер: нет их в PR. Значит после мержа рюкзаки скорее всего будут без спрайта при надевании на спину.

3. 🗑️ Сэтчел потерял icon.png, inhand-left.png, inhand-right.png
Все три удалены и заменены файлами с нестандартными именами (left hand.png, right hand.png). RSI meta.json при этом тоже не обновлён в PR — значит новые стейты вообще не зарегистрированы.

4. 📋 meta.json не тронуты ни в одном из трёх .rsi
Новые файлы добавлены, но записи в meta.json под них — нет. Движок просто проигнорирует эти спрайты.


В целом — арт, судя по описанию, красивый, но технически PR сыроват. Поправь имена файлов (без пробелов, по конвенции), обнови meta.json во всех трёх RSI и проверь YAML-определения на equipped-BACKPACKequipped. Тогда будет огонь 🔥

( ˘ ³˘)♥ 🐇

@RJD1TOP1

Copy link
Copy Markdown
Contributor Author

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain
🏁 Script executed:

#!/bin/bash
gh pr view 1057 --repo space-sunrise/project-fire --json files,additions,deletions,changedFiles | jq .
gh pr diff 1057 --repo space-sunrise/project-fire --name-only

Length of output: 5381

@RJD1TOP1: Ладно, держись — ты сам попросил 🐇

Официальный разбор полётов от пушистого ревьюера:

1. 🚨 Пробелы в именах файлов — серьёзно??

  • open b.png — что такое "b"? Открытый "б"? Это не имя файла, это опечатка.
  • left hand.png, right hand.png — пробелы в именах файлов в RSI это билет в однострочный баг-репорт. Стандарт проекта: inhand-left.png / inhand-right.png. Уже было в остальных папках, но для сэтчела решили поизобретать велосипед.

2. 💀 equipped-BACKPACK.pngequipped.png везде Удалил старый стейт, добавил новый с другим именем — а YAML-определения сущностей обновил? Спойлер: нет их в PR. Значит после мержа рюкзаки скорее всего будут без спрайта при надевании на спину.

3. 🗑️ Сэтчел потерял icon.png, inhand-left.png, inhand-right.png Все три удалены и заменены файлами с нестандартными именами (left hand.png, right hand.png). RSI meta.json при этом тоже не обновлён в PR — значит новые стейты вообще не зарегистрированы.

4. 📋 meta.json не тронуты ни в одном из трёх .rsi Новые файлы добавлены, но записи в meta.json под них — нет. Движок просто проигнорирует эти спрайты.

В целом — арт, судя по описанию, красивый, но технически PR сыроват. Поправь имена файлов (без пробелов, по конвенции), обнови meta.json во всех трёх RSI и проверь YAML-определения на equipped-BACKPACKequipped. Тогда будет огонь 🔥

( ˘ ³˘)♥ 🐇

Ты довёл меня до сердечного приступа.

Я требую конпенсации.
@CodeRabbit

@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Tip

For best results, initiate chat on the files or code changes.

@RJD1TOP1 Компенсация принята к рассмотрению 🐇

Условия выплаты:

  1. Переименовать left hand.pnginhand-left.png, right hand.pnginhand-right.png
  2. Обновить meta.json во всех трёх RSI
  3. Проверить YAML на equipped-BACKPACKequipped

После выполнения пунктов 1–3 долг считается погашен, кардиограмма выравнивается, и спрайты от пепера выходят в прод 🔥

Ты сам попросил разбор — кролик лишь исполнил долг. Желаю скорейшего выздоровления и успешного фикса! ヾ(^ᴗ^)ノ

(´;ω;`) 🐇

@WardexOfficial

Copy link
Copy Markdown
Collaborator
image Чини

@WardexOfficial

Copy link
Copy Markdown
Collaborator

@RJD1TOP1
Нужно что бы ты установил цифровую подпись своих коммитов на гитхабе (инструкция: https://emmanuelbernard.com/blog/2023/11/27/git-signing-ssh/) и вероятнее всего либо пересоздал все коммиты сам либо воспользовался rebase для подписи всех коммитов

@RJD1TOP1

Copy link
Copy Markdown
Contributor Author

@RJD1TOP1 Нужно что бы ты установил цифровую подпись своих коммитов на гитхабе (инструкция: https://emmanuelbernard.com/blog/2023/11/27/git-signing-ssh/) и вероятнее всего либо пересоздал все коммиты сам либо воспользовался rebase для подписи всех коммитов

Зачем это

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants