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: Detective zippo #6284

Merged
merged 7 commits into from
Dec 30, 2024
Merged

add: Detective zippo #6284

merged 7 commits into from
Dec 30, 2024

Conversation

FlitchTime
Copy link
Contributor

@FlitchTime FlitchTime commented Dec 29, 2024 •
edited
Loading

Описание

Создаём новую Зипку для детектива и суём её на основные карты в его кабинет

Причина создания ПР / Почему это хорошо для игры

Прикол
Оказывается двойной прикол

Демонстрация изменений

image

Тесты

Локалка

@FlitchTime FlitchTime requested a review from a team as a code owner December 29, 2024 16:05
@github-actions github-actions bot added MAP Affects map DMI Affects DMI files labels Dec 29, 2024
PlayerUnknown14
PlayerUnknown14 requested changes Dec 30, 2024
View reviewed changes
Copy link
Member

@PlayerUnknown14 PlayerUnknown14 left a comment

Choose a reason for hiding this comment

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

Вето на ведро от главлока. Слегка поправить надобно.

code/game/objects/items/weapons/lighters.dm Outdated Show resolved Hide resolved
@FlitchTime
Copy link
Contributor Author

Собираю апрувы, любой ценой, но бесплатно

PlayerUnknown14
PlayerUnknown14 previously approved these changes Dec 30, 2024
View reviewed changes
Copy link
Member

@PlayerUnknown14 PlayerUnknown14 left a comment

Choose a reason for hiding this comment

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

Апрув на ведро от главлока.

@BeebBeebBoob
Copy link
Member

Надо предложку делать

@BeebBeebBoob BeebBeebBoob added the требует голосовалку Всё ещё проходит голосовалки, либо ещё нету самой предложке в книге. label Dec 30, 2024
@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Dec 30, 2024
@FlitchTime FlitchTime closed this Dec 30, 2024
@FlitchTime FlitchTime reopened this Dec 30, 2024
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Dec 30, 2024
@pilygun
Copy link
Contributor

pilygun commented Dec 30, 2024

Надо предложку делать

Дядя с головой все в порядке? Во первых на три из 6 ПРов на которые ты значок "[требует голосовалку" НА НИХ СУКА БЫЛА ПРЕДЛОЖКА, я помню за каждую голосовал блять. А даже если бы и не было, это настолько минорное изменение. Это действительно требует предложки? Это спрайт от ведущего спрайтера блять, они априори должны без предложек мержится, ибо это стандарт качества спрайтов. Ебана рот столько с цифровой бюрократией боролись, пытались отказаться от системы воутов, а бипу похуй.
Чесно я немного хуею, я думал если в дискорде пиздец, то хотя бы на гитхабе все ок будет.

@pilygun pilygun removed the требует голосовалку Всё ещё проходит голосовалки, либо ещё нету самой предложке в книге. label Dec 30, 2024
pilygun
pilygun approved these changes Dec 30, 2024
View reviewed changes
Copy link
Contributor

@pilygun pilygun left a comment

Choose a reason for hiding this comment

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

предложка была

@BeebBeebBoob
Copy link
Member

Я когда проверял ПР, тут не было ссылки на предложку. Отсюда очевидный вывод: не было предложки.
Это не в моих интересах идти и искать на этот ПР предложку. Если вы не можете скопировать и вставить одну ссылку в описание, то вам стоит пересмотреть своё желание делать ПРы.

Даже столь мелкое и минорное добавление требует предложку. Да, люди настолько чувствительные что могут отказать тебе в добавлении какой либо косметики в игру.

Напоследок, следи за порывами своих эмоции. Не нравиться - не ешь.

@BeebBeebBoob BeebBeebBoob merged commit 8e0d1f4 into ss220-space:master220 Dec 30, 2024
12 checks passed
@pilygun
Copy link
Contributor

pilygun commented Dec 31, 2024

8

@PiotrTheTchaikowsky
Copy link
Contributor

image

@FlitchTime FlitchTime deleted the zippo-for-detective branch December 31, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pilygun pilygun pilygun approved these changes

@PlayerUnknown14 PlayerUnknown14 PlayerUnknown14 left review comments

@Antoonij Antoonij Awaiting requested review from Antoonij

Assignees
No one assigned
Labels
DMI Affects DMI files MAP Affects map
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@FlitchTime @BeebBeebBoob @pilygun @PiotrTheTchaikowsky @Antoonij @PlayerUnknown14