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

minesweeper #1071

Merged
merged 34 commits into from
Feb 4, 2025
Merged

minesweeper #1071

merged 34 commits into from
Feb 4, 2025

Conversation

ROdenFL
Copy link

@ROdenFL ROdenFL commented Jan 26, 2025 •
edited by sourcery-ai bot
Loading

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

Добавляет сапёр в пда. Емаг будет взрывать вас при проигрыше. В бд будут сохраняться результаты побед.

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

{DC5D7E08-B1DE-415F-A654-E66524F3E8D6}

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

{09FED18B-CB10-439D-834D-83D7FE4872D0}

{2D10A433-4B2D-42A1-A5CA-D735BC5DBD75}

{006EE856-9440-47A9-AC4F-67BA57DAFF5A}

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

image

Changelog

🆑
add: Сапёр в пда
/:cl:

Summary by Sourcery

Add Minesweeper to PDAs. Emagged PDAs will explode upon losing the game. Game results are saved to the database.

New Features:

  • A Minesweeper game has been added to PDAs.

Tests:

  • Added playtesting screenshots.

@sourcery-ai Sourcery AI

This comment was marked as spam.

Sign in to view
@github-actions github-actions bot added the TGUI Добавление или изменение существующего интерфейса на базе фреймворка TGUI label Jan 26, 2025
@Lime-7
Copy link

Lime-7 commented Jan 26, 2025

Почему это хорошо:
... ... ...

@PhantornRU
Copy link

При емаге/взломе нужен функционал подрыва ПДА, если попал на бомбу.

@github-actions github-actions bot added :neckbeard: Документация Некоторые просто любят писать тонну текста, которую никто не будет читать. 💿 SQL Надо править БД. labels Jan 29, 2025
@ROdenFL
Copy link
Author

ROdenFL commented Jan 29, 2025

@sourcery-ai title

@ROdenFL
Copy link
Author

ROdenFL commented Jan 29, 2025

работай давай

@ROdenFL
Copy link
Author

ROdenFL commented Jan 29, 2025

@sourcery-ai review

@ROdenFL ROdenFL marked this pull request as ready for review January 29, 2025 18:28
@ss220app ss220app bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Jan 29, 2025
Gaxeer
Gaxeer requested changes Jan 30, 2025
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.

Пока так, первичное ревью.

modular_bandastation/minesweeper/code/minesweeper.dm Outdated Show resolved Hide resolved
modular_bandastation/minesweeper/code/minesweeper.dm Outdated Show resolved Hide resolved
modular_bandastation/minesweeper/code/minesweeper.dm Show resolved Hide resolved
@ROdenFL ROdenFL marked this pull request as draft February 1, 2025 08:07
@ROdenFL ROdenFL marked this pull request as ready for review February 1, 2025 12:50
Gaxeer
Gaxeer requested changes Feb 2, 2025
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.

Посмотри, протести, скажи если я ошибаюсь.

modular_bandastation/minesweeper/code/minesweeper.dm Outdated Show resolved Hide resolved
modular_bandastation/minesweeper/code/minesweeper.dm Outdated Show resolved Hide resolved
ROdenFL and others added 2 commits February 3, 2025 10:55
@ROdenFL @Gaxeer
Update modular_bandastation/minesweeper/code/minesweeper.dm
3e12362 
Co-authored-by: Gaxeer <44334376+Gaxeer@users.noreply.github.com>
@ROdenFL @Gaxeer
Update modular_bandastation/minesweeper/code/minesweeper.dm
b1cd064 
Co-authored-by: Gaxeer <44334376+Gaxeer@users.noreply.github.com>
@Legendaxe Legendaxe merged commit c1dda0d into ss220club:master Feb 4, 2025
23 checks passed
ss220app bot added a commit that referenced this pull request Feb 4, 2025
@ss220app
Automatic changelog for PR #1071 [ci skip]
fa47895
@ROdenFL
Copy link
Author

ROdenFL commented Feb 4, 2025

Ну либо не сделаю

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Furrior Furrior Furrior left review comments

@Legendaxe Legendaxe Legendaxe approved these changes

@Gaxeer Gaxeer Gaxeer approved these changes

Assignees
No one assigned
Labels
💿 SQL Надо править БД. :neckbeard: Документация Некоторые просто любят писать тонну текста, которую никто не будет читать. 📜 CL валиден Этот чейнджлог будет успешно опубликован TGUI Добавление или изменение существующего интерфейса на базе фреймворка TGUI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ROdenFL @Lime-7 @PhantornRU @Gaxeer @Furrior @Legendaxe @AyIong