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

ADD: angel suit NT #1087

Open
wants to merge 12 commits into
base: beta-dev
Choose a base branch
from
Open

ADD: angel suit NT #1087

wants to merge 12 commits into from

Conversation

SantaKO1
Copy link

@SantaKO1 SantaKO1 commented Oct 17, 2024 •
edited
Loading

ПР делает что-то по типу добавления хардсьютов для НТ.

  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.

@MysticalFaceLesS MysticalFaceLesS added 🎨 Спрайты 🎨 Есть спрайты 🧱 Код 🧱 Изменения .dm файлов 🔵 Требуется проверка 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! labels Oct 17, 2024
@SantaKO1 SantaKO1 changed the title nasral ADD: angel suit NT Oct 17, 2024
@MrCat15352 MrCat15352 added ⌛ Ждёт результатов голосования ⌛ PR ждет результатов голосования по предложке в дискорде ❗ PR НЕ МЁРЖИТЬ ❗ PR НЕ МЁРЖИТЬ 🪓 НЕ МОДУЛЬНО 🪓 Не мудульно да, переделать бы 💢 Плохой PR 💢 Плохое оформление ПРа labels Oct 18, 2024
@MrCat15352
Copy link
Collaborator

Плюсом, отсутствует включение файла пака в главном файле модуля. Ваш пак может вызвать ошибки кода.

@arealbeba
Copy link

фиксим

@MysticalFaceLesS MysticalFaceLesS removed the 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! label Oct 18, 2024
MrCat15352
MrCat15352 requested changes Oct 18, 2024
View reviewed changes
mod_celadon/nt_hardsuit/README.md Outdated Show resolved Hide resolved
MrCat15352
MrCat15352 requested changes Oct 18, 2024
View reviewed changes
mod_celadon/mod_celadon.dme Outdated Show resolved Hide resolved
@arealbeba
Copy link

фиксики работают как часы

@MrCat15352 MrCat15352 added 🧻 Документация 🧻 Что-то связаное с документацией and removed 💢 Плохой PR 💢 Плохое оформление ПРа labels Oct 18, 2024
@MrCat15352 MrCat15352 added 🔊 Звуки 🔊 Изменения звуков 🏆 Отличная работа! 🏆 PR как и сама работа, оформленыи сделаны отлично! 🔆 Обновление 🔆 PR содержит или добавляет что-то новое или старое, или обновляет... labels Oct 18, 2024
@SantaKO1 SantaKO1 marked this pull request as draft October 18, 2024 12:52
@SantaKO1 SantaKO1 marked this pull request as ready for review October 20, 2024 14:29
@MrCat15352 MrCat15352 added 🛠 В разработке 🛠 В драфте, в разработке. До лучших времен and removed ⌛ Ждёт результатов голосования ⌛ PR ждет результатов голосования по предложке в дискорде labels Oct 24, 2024
@MrCat15352
Copy link
Collaborator

ЗАМОРОЖЕНО ПО ПРИКАЗУ ВЫСШЕГО ПАЛАЧА

@github-actions GitHub Actions
Copy link

Pull Request содержит конфликты. Устраните их, чтобы мы смогли оценить и принять запрос.

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

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

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

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
Merge Conflict ❗ PR НЕ МЁРЖИТЬ ❗ PR НЕ МЁРЖИТЬ 🎨 Спрайты 🎨 Есть спрайты 🏆 Отличная работа! 🏆 PR как и сама работа, оформленыи сделаны отлично! 🔆 Обновление 🔆 PR содержит или добавляет что-то новое или старое, или обновляет... 🔊 Звуки 🔊 Изменения звуков 🔵 Требуется проверка 🚧 Долгострой 🚧 PR не мержить. Долгоиграющий. 🛠 В разработке 🛠 В драфте, в разработке. До лучших времен 🧱 Код 🧱 Изменения .dm файлов 🧻 Документация 🧻 Что-то связаное с документацией 🪓 НЕ МОДУЛЬНО 🪓 Не мудульно да, переделать бы
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@SantaKO1 @MrCat15352 @arealbeba @MysticalFaceLesS