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

Новый эвей мишион, и чуть чуть для ерт там #43

Closed
wants to merge 10 commits into from

Conversation

lobzik8
Copy link

@lobzik8 lobzik8 commented Apr 20, 2024

No description provided.

@github-actions github-actions bot added the Code label Apr 20, 2024
Copy link
Member

Choose a reason for hiding this comment

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

Требуется вынести в модуль. Либо частично, либо полностью. Лишнее не удалить, а закомментировать /* */

@github-actions github-actions bot removed the Code label Apr 21, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate your pull request.

@github-actions github-actions bot added the Code label Apr 27, 2024
@MrCat15352 MrCat15352 added 🚨Не мёржить!🚨 По каким-то причинам не мержить PR 🔎 Требует рассмотрения 🔍 🧱 Код 🧱 В PR есть код... and removed Code labels Apr 28, 2024
@github-actions github-actions bot added the Code label Apr 29, 2024
Copy link

github-actions bot commented May 3, 2024

This pull request has conflicts, please resolve those before we can evaluate your pull request.

@MrCat15352 MrCat15352 added ⚠Конфликты⚠ В PR есть конфликты. Исправить надо бы 🛠 Доработать 🛠 PR нужно доработать. Есть комментарии или нет их... and removed Code Merge Conflict labels May 4, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

Собственно, оформить нужно как в ридми mod_celadon написано. Добавить что ты сделал, айди поставить, закомментить оригинальный код. А собственно что именно тут ты сделал? @lobzik8

Copy link
Collaborator

Choose a reason for hiding this comment

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

Весь кор код комментить теперь не надо. И что тут ты менял? @lobzik8

Copy link
Collaborator

Choose a reason for hiding this comment

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

У нас карты переместились. Тут надо решить конфликт тебе

@@ -9,5 +9,5 @@
#include "code/security_state.dm"
#include "code/mob_helpers.dm"
#include "code/cards_ids.dm"

#endif
Copy link
Collaborator

Choose a reason for hiding this comment

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

И на кой тебе черт удалять это? @lobzik8

Copy link
Collaborator

Choose a reason for hiding this comment

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

Опять же, у меня ВОПРОСЫ, ты менял тут ПРЯМ ВСЁ? @lobzik8

Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично вышему

@MrCat15352 MrCat15352 added the 📢Не модульно! Код не соответствует требованиям модульности label May 4, 2024
Copy link

github-actions bot commented May 4, 2024

This pull request has conflicts, please resolve those before we can evaluate your pull request.

@MrCat15352 MrCat15352 added the ❌ Отказано ❌ PR закрыт label May 11, 2024
@MrCat15352 MrCat15352 closed this May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict ⚠Конфликты⚠ В PR есть конфликты. Исправить надо бы ❌ Отказано ❌ PR закрыт 📢Не модульно! Код не соответствует требованиям модульности 🔎 Требует рассмотрения 🔍 🚨Не мёржить!🚨 По каким-то причинам не мержить PR 🛠 Доработать 🛠 PR нужно доработать. Есть комментарии или нет их... 🧱 Код 🧱 В PR есть код...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants