Skip to content

UPDATE: Обновление расы Таяран, а не таяра #2259

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 14 commits into
base: beta-dev
Choose a base branch
from
Draft

UPDATE: Обновление расы Таяран, а не таяра #2259

wants to merge 14 commits into from
+1,677 −1,332

Conversation

MrCat15352
Copy link

Описание / Что этот PR делает

Причина создания ПР / Почему это хорошо для игры

Демонстрация изменений / Тестирование

Список изменений

🆑
add: Что-то добавлено
del: Что-то удалено
balance: Что-то изменено в балансе
tweak: Что-то изменено по мелочи
fix: Что-то исправлено
sound: Добавлен/Изменен/Удален звук
image: Добавлен/Изменен/Удален спрайт
map: Изменения на карте
spellcheck: Исправлена опечатка/добавлен перевод
admin: Изменения в кнопках администрации
/🆑

@MrCat15352
Добавляем новые спарйты, частично заменяя их новыми
4261b96
@MrCat15352
Merge remote-tracking branch 'origin/combocat' into update_tajaran
7085fb3
@MrCat15352
Рефактор название ИД с таяры на таяран как вид
22cefc9
@MrCat15352
Рефактор, снова переименовываем, добавяля пропущенную буковку n
b8bc5ee
@MrCat15352
Рефакторинг, приводим в правильный вид код таяран
8e73e42
@MrCat15352
Рефакторинг. Переименовка файлов, о да!
e4ca88c
@MrCat15352
Рефакторинг. Больше изменений! Привели в порядок все!
e1687a0
@MrCat15352
Рефакторинг названий спрайтов
5adf105
@MrCat15352
Добавление новых видов хвостов
b756a79
@MrCat15352
Фикс спрайтов. Перепутались два вида
d2f8261
@MrCat15352
Merge branch 'CeladonSS13:beta-dev' into update_tajaran
aac1ec1
@Celadon-Bot Celadon-Bot added 💬 TGUI Кто-то решил потрогать интерфейсы 🗺 Маппинг 🗺 Это чисто карты, карта... 🎨 Спрайты 🎨 Есть спрайты 🧱 Код 🧱 Изменения .dm файлов 🔮 Конфиги 🔮 Конфиги изменены.. Но зачем? 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! 🏔 Руинки 🏔 Изменение карт с руинами labels Sep 17, 2025
@github-actions github-actions bot added the 🔥 Merge Conflict 🔥 Pull Request содержит конфликты. Устраните их, чтобы мы смогли оценить и принять запрос. label Sep 17, 2025
@github-actions GitHub Actions

This comment was marked as resolved.

Sign in to view
KOCMODECAHTHUK
KOCMODECAHTHUK requested changes Sep 17, 2025
View reviewed changes
Copy link
Member

@KOCMODECAHTHUK KOCMODECAHTHUK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Пути не трогай пусть будут с ошибкой.
Разбей ПР на несколько, никто в здравом уме не будет рассматривать множественное изменение кода и не только.

@KOCMODECAHTHUK
Copy link
Member

KOCMODECAHTHUK commented Sep 17, 2025 •
edited
Loading

Рефакторинг ради 1 буквы того не стоит, очень высока вероятность ВСЕ сломать.

  • Очень сомнительное изменение путей
  • Их никто не будет так называть, все уже привыкли к старому названию (Таяр/Таяра)
  • Ты сломаешь игрокам пресеты персонажей

У тебя буквально игроки взвоют из-за того что ты им персонажей сломаешь с пресетами их настроек.
И насчет спрайтов, если диги-ног:

  • Только если будут соответствовать стандартам под текущую одежду в билде.

@MrCat15352
Merge branch 'beta-dev' into update_tajaran
13d7bd3 
Signed-off-by: MrCat15352 <125268716+MrCat15352@users.noreply.github.com>
@github-actions github-actions bot removed the 🔥 Merge Conflict 🔥 Pull Request содержит конфликты. Устраните их, чтобы мы смогли оценить и принять запрос. label Sep 18, 2025
@MrCat15352
Copy link
Author

Миграция написана и ждет пока гитхаб поднимется

@MrCat15352 MrCat15352 added the 🚧 Долгострой PR не мержить. Долгоиграющий. label Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cuildipie Cuildipie Awaiting requested review from Cuildipie Cuildipie is a code owner

@Kenfarka Kenfarka Awaiting requested review from Kenfarka Kenfarka is a code owner

@MysticalFaceLesS MysticalFaceLesS Awaiting requested review from MysticalFaceLesS MysticalFaceLesS is a code owner

@Vairkharst Vairkharst Awaiting requested review from Vairkharst Vairkharst is a code owner

@KOCMODECAHTHUK KOCMODECAHTHUK Awaiting requested review from KOCMODECAHTHUK KOCMODECAHTHUK is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
🎨 Спрайты 🎨 Есть спрайты 🏔 Руинки 🏔 Изменение карт с руинами 💬 TGUI Кто-то решил потрогать интерфейсы 🔮 Конфиги 🔮 Конфиги изменены.. Но зачем? 🗺 Маппинг 🗺 Это чисто карты, карта... 🚧 Долгострой PR не мержить. Долгоиграющий. 🧱 Код 🧱 Изменения .dm файлов 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MrCat15352 @KOCMODECAHTHUK @Celadon-Bot