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.

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

Diona Pacifist fix #96

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Diona Pacifist fix #96

merged 1 commit into from
Jul 17, 2023

Conversation

aw-c
Copy link

@aw-c aw-c commented Jun 30, 2023 •
edited
Loading

Описание PR

Пацифизим должен быть выбором каждой дионы отдельно (благо кнопка есть), а не чертой целой расы!

Проверки

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

Изменения
🆑 Alan Wake

  • tweak: Убран обязательный пацифизм у дион.

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Jun 30, 2023
Legendaxe
Legendaxe approved these changes Jun 30, 2023
View reviewed changes
Copy link
Collaborator

@Legendaxe Legendaxe left a comment

Choose a reason for hiding this comment

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

Вообще мне нравился принудительный пацифизм как особенность расы, но решать не мне.
Аппруваю

@aw-c aw-c merged commit aa485f0 into SerbiaStrong-220:master Jul 17, 2023
@aw-c aw-c deleted the second_task branch July 17, 2023 18:35
@Grigew
Copy link

Grigew commented Jul 18, 2023

Не канон нужно заревертить

@Grigew Grigew mentioned this pull request Jul 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Legendaxe Legendaxe Legendaxe approved these changes

Assignees
No one assigned
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps"
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aw-c @Grigew @Legendaxe