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

Чемоданчик #11398

Conversation

DarthSidiousPalpatine
Copy link
Contributor

Описание изменений

pri-ugroze-terroristicheskogo-akta_1610977617926912112

Почему и что этот ПР улучшит

Отучит разминировать лицом сумки, а также вообще лезть в бесхозные сумки.

Авторство

AndreyGysev и все все все, кто помогал в дискорде.

Чеинжлог

🆑

  • rscadd: Добавлен рандомраундстартивент на появление сумки в случайном месте станции. В сумке может быть мина или обычные предметы.

@TauKitty
Copy link
Contributor

TauKitty commented May 8, 2023

Changelog status: ✔️

@TauKitty TauKitty added the Feature Новая фича label May 8, 2023
@TauKitty
Copy link
Contributor

TauKitty commented May 8, 2023

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/fludilka-chast-2/28659/23153

Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield left a comment

Choose a reason for hiding this comment

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

мне не нрав как реализован рандомизатор

я бы предпочёл рандом/кошелёк, рандом/сумочка, и потом рандом/сторедж

почти везде у нас иерархия именно так и работает

@DarthSidiousPalpatine
Copy link
Contributor Author

Сделал.

@DarthSidiousPalpatine
Copy link
Contributor Author

Вот тут из требований - ты хочешь особую гранату.

@@ -227,7 +264,7 @@
)
new itemtype(Bag)

/obj/random/misc/storage/mine
/obj/random/misc/mine
Copy link
Contributor

Choose a reason for hiding this comment

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

выглядит будто это рандомная мина а не коробка с миной

Copy link
Contributor Author

Choose a reason for hiding this comment

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

так на картинке с ней будет сумка. так что явно будет понятно что это рандомная коробка с миной

Copy link
Contributor

Choose a reason for hiding this comment

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

названия важны. в названии написано что это мина а не сумка с миной. напиши что это сумка с миной

var/obj/item/weapon/storage/Bag = ..()

if(prob(30))
new /obj/item/weapon/grenade/chem_grenade/mine(Bag)
Copy link
Contributor

Choose a reason for hiding this comment

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

в чем прикол больше одной гранаты

Copy link
Contributor Author

Choose a reason for hiding this comment

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

5 гранат по 30 ломают человека но не ломают тайлы.

Copy link
Contributor

Choose a reason for hiding this comment

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

а 1 граната не ломает?

Copy link
Contributor

Choose a reason for hiding this comment

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

ну и в любом случае заменить на

for(var/i in 1 to количество)
    new граната

/obj/random/misc/mine/backpack_satchel/spawn_item()
var/obj/item/weapon/storage/Bag = ..()

if(prob(30))
Copy link
Contributor

Choose a reason for hiding this comment

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

тут и везде переписать на вид

if(prob(30))
    ...
    return

for ...
    ...

(т.н. "ранний возврат")

@github-actions
Copy link

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@github-actions
Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Jun 20, 2023
@DarthSidiousPalpatine
Copy link
Contributor Author

Как бы я хотел это вернуть и довести до мержа, Людук приди.

@DarthSidiousPalpatine
Copy link
Contributor Author

Так, ща тогда проверю сделал ли я то о чём ты тут написал и не скрыл и спрошу что дальше.

@DarthSidiousPalpatine
Copy link
Contributor Author

Спасибо воласу.

@DarthSidiousPalpatine
Copy link
Contributor Author

Волас поменял взрывы и теперь какой бы бомбу и в скольких количествах не делал не добиться гиба руки при сохранении тайла под челом

Волас поменял полностью код рандом предметов и теперь я не могу подтягиваться внутрь прока после спавна предмета и пихать в сумку лут или гранаты

Волас требует написать, блять, свой, параллельный тип рандомных, блять, предметов, просто потому что его тип не возвращает предмет что он спавнит. Ахуенно.

Да нахуй это всё, блять, заебись фича была, немного доп. работки сбухам, разминирование, всё такое. Замержили бы раньше - Волас бы со всем этим ебался, ну или удалил бы фичу. А я не буду.

@volas
Copy link
Member

volas commented Aug 17, 2023

  1. Нечего браться за десяток ПРов и не доводить их до мержа, ждать конфлитков.
  2. Нужно читать и слушать, что я пишу, я всё объяснил и даже накидал вариант и предложил помощь.
  3. По взрывам есть ишью и обсуждения на форуме, я открыт для фидбека.
  4. Я бы даже мог из принципа доделать всё сам за тебя, что бы показать, насколько ты тормоз, но мне не нравится фича и других профитов нет.

@DarthSidiousPalpatine
Copy link
Contributor Author

Даа дааа, конечно.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants