Skip to content
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.

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

Maior rxbzz #137

Closed
wants to merge 6 commits into from
Closed

Maior rxbzz #137

wants to merge 6 commits into from

Conversation

RipZoro1
Copy link
Contributor

@RipZoro1 RipZoro1 commented Jul 26, 2024

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения
no cl, перенос Майора РХБЗЗ

@github-actions github-actions bot added No C# Changes: Localization Изменение локализации labels Jul 26, 2024
@RipZoro1
Copy link
Contributor Author

блятьб, я делал на сборке, которая поломалась. так что проверить не могу :(

Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment

Choose a reason for hiding this comment

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

Описание пра? Что здесь добавляется? Я как по коду смотрю тут и другая херня есть
Напиши конкретно что именно и добавь то что относится конкретно к этому обновлении(что за руны?Что за Event-Cult-of-the-Lamb) без этого ревью нельзя нормально провести

@github-actions github-actions bot added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Jul 27, 2024
@Schrodinger71 Schrodinger71 linked an issue Aug 7, 2024 that may be closed by this pull request
- type: dataset
id: ADTCmndNamesFirstMilitaryCorvax
values:
- Майор
Copy link
Contributor

Choose a reason for hiding this comment

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

Майор? Просто Майор? добавивь вариативность имен. Например Майор Оруэл.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Это первая часть приписки, всё ок, а ВОТ К НАЗВАНИЮ АЙДИШНика вопросы, с хера корвакс Corvax
image


#START-Event-Cult-of-the-Lamb
- type: entity
name: Cultist
Copy link
Contributor

Choose a reason for hiding this comment

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

Это что за ивент. Этого тут быть не должно как и того что ниже


#СССП штуки
#Рядовой
- type: entity
Copy link
Contributor

Choose a reason for hiding this comment

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

СССП штуки? Мы же переносим Майора

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации No C# Status: Awaiting Changes Ожидание изменений
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants