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

Tweak: Update railings sprite #676

Merged
merged 2 commits into from
Nov 12, 2023
Merged

Tweak: Update railings sprite #676

merged 2 commits into from
Nov 12, 2023

Conversation

dj-34
Copy link
Collaborator

@dj-34 dj-34 commented Nov 12, 2023

Что этот PR делает

Порт обновленного спрайта перил с ТГ + свои правки в спрайт, чтобы стыковка выглядела нормально

Почему это хорошо для игры

Более симпатично

Изображения изменений

railing
image

Тестирование

Ну в игрушке посидел попердел

Changelog

🆑
tweak: Обновление спрайта перил
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 12, 2023
@konushi
Copy link

konushi commented Nov 12, 2023

как же сочно выглядит

Gaxeer
Gaxeer approved these changes Nov 12, 2023
View reviewed changes
Copy link
Collaborator

@Gaxeer Gaxeer left a comment

Choose a reason for hiding this comment

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

Красивые. Добавляют обьема.

@dj-34 dj-34 marked this pull request as draft November 12, 2023 00:48
AyIong
AyIong previously approved these changes Nov 12, 2023
View reviewed changes
Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

LGTM

@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

Нужно мне карты протыкать, чтобы на всех нормально выглядело...

@AyIong
Copy link
Collaborator

AyIong commented Nov 12, 2023

Не думаю что где-то будут проблемы

@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

Не думаю что где-то будут проблемы

Увы, но они есть

@github-actions github-actions bot added 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) labels Nov 12, 2023
@dj-34 dj-34 requested a review from AyIong November 12, 2023 01:17
@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

Пофиксил основное, найти и просмотреть все косяки (если они есть), к сожалению, не в состоянии
image

@dj-34 dj-34 marked this pull request as ready for review November 12, 2023 01:18
@dj-34 dj-34 marked this pull request as draft November 12, 2023 01:24
@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

хотя пробегусь еще раз сегодня вечером

@AyIong
Copy link
Collaborator

AyIong commented Nov 12, 2023

Ну ты бессмертный

@AyIong
Copy link
Collaborator

AyIong commented Nov 12, 2023

Какни ПЖ на мету чёт по углам карты, шоб было видно в диффботе чё там подредачить надо будет

@AyIong AyIong dismissed their stale review November 12, 2023 01:31

Мы ОТВЕТСТВЕННЫЕ, так шо без ревью галки не ставим :AgaAga:

@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

Порядок, достаточно было просто скипнуть Цере и не так уж и много нужно было зачекать
image

Всё в рамках нормы, если не считать немного шизы от оффов в виде двойных перил (не критичино)
image

@dj-34 dj-34 marked this pull request as ready for review November 12, 2023 01:35
@AyIong
Copy link
Collaborator

AyIong commented Nov 12, 2023

На мете порядок?

@dj-34
Copy link
Collaborator Author

dj-34 commented Nov 12, 2023

На мете порядок?

да

AyIong
AyIong approved these changes Nov 12, 2023
View reviewed changes
Copy link
Collaborator

@AyIong AyIong left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Заебца

@AyIong AyIong merged commit c4b77d9 into ss220club:master Nov 12, 2023
15 checks passed
@dj-34 dj-34 deleted the new_railing_sprite branch December 1, 2023 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AyIong AyIong AyIong approved these changes

@Gaxeer Gaxeer Gaxeer approved these changes

Assignees
No one assigned
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@dj-34 @konushi @AyIong @Gaxeer @ss220bot