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

Disposal Unit AI interactible fix #2213

Merged
merged 12 commits into from
Nov 6, 2024
Merged

Disposal Unit AI interactible fix #2213

merged 12 commits into from
Nov 6, 2024

Conversation

Bomjojuk
Copy link
Collaborator

@Bomjojuk Bomjojuk commented Nov 5, 2024

Описание PR

Добавляет возможность станционному ИИ взаимодействовать с мусорками

Проверки

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

Изменения
🆑

  • tweak: Станционный ИИ теперь может взаимодействовать с утилизационными блоками

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Nov 5, 2024
@Bomjojuk Bomjojuk requested a review from stalengd November 5, 2024 21:18
@stalengd stalengd requested a review from Ady4ik November 5, 2024 21:35
@stalengd stalengd added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Nov 5, 2024
@UrPrice UrPrice removed the request for review from Ady4ik November 5, 2024 21:37
@UrPrice
Copy link
Collaborator

UrPrice commented Nov 5, 2024

@stalengd без ревью Адуча, минорные изменения можем без его апрува рассматривать

@stalengd
Copy link
Collaborator

stalengd commented Nov 5, 2024

@stalengd без ревью Адуча, минорные изменения можем без его апрува рассматривать

Вот только кто без ревью определит что они минорные...
Ладно, если это минорное, то окей

@UrPrice
Copy link
Collaborator

UrPrice commented Nov 5, 2024

@stalengd без ревью Адуча, минорные изменения можем без его апрува рассматривать

Вот только кто без ревью определит что они минорные... Ладно, если это минорное, то окей

Ну типа трекероведов на это и набирали, чтобы они могли пушить минорные изменения без излишней бюрократии.

@Bomjojuk
Copy link
Collaborator Author

Bomjojuk commented Nov 5, 2024

@stalengd без ревью Адуча, минорные изменения можем без его апрува рассматривать

Вот только кто без ревью определит что они минорные... Ладно, если это минорное, то окей

то что сказал прайс, + мы это уже обсудили в идеях qa

stalengd
stalengd requested changes Nov 5, 2024
View reviewed changes
Copy link
Collaborator

@stalengd stalengd left a comment

Choose a reason for hiding this comment

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

А, стоп, а метку?

@Bomjojuk
Copy link
Collaborator Author

Bomjojuk commented Nov 5, 2024

А, стоп, а метку?

Точн, сек

@Surani1
Copy link

Surani1 commented Nov 6, 2024

12 коммитов и изменение одной строчки
155774f070222683c5730fce24794328

@stalengd stalengd merged commit 1e248d6 into SerbiaStrong-220:master Nov 6, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stalengd stalengd stalengd approved these changes

Assignees
No one assigned
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Need to be discussed Требуется обсуждение перед аппрувом/мёрджем Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Bomjojuk @UrPrice @stalengd @Surani1 @Ady4ik