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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gives CMO a syringe gun in their locker #17342

Merged
merged 1 commit into from Jan 10, 2023

Conversation

Moltijoe
Copy link
Contributor

@Moltijoe Moltijoe commented Jan 5, 2023

Security having exclusive access to the syringe guns is weird considering only medical can really use them.
It being locked to CMO means medical still has access if something goes wrong, but shouldn't allow for easy validhunting.

馃啈
rscadd: CMO gets a syringe gun in their locker
/:cl:

@Yogbot-13 Yogbot-13 added the Feature This adds new content to the game label Jan 5, 2023
@Moja10
Copy link
Contributor

Moja10 commented Jan 6, 2023

Don't enable that shit PR, it should be reverted outright

@MajManatee
Copy link
Contributor

Moja do be having a point tho

@Hopekz
Copy link
Contributor

Hopekz commented Jan 7, 2023

Security having exclusive access to the syringe guns is weird considering only medical can really use them.

wdym with #17276 security can now fire lethal execution syringes at people.. which is kind of the logic of #17276 which you merged..?

The syringe gun in the armory spawns with execution syringes
image

@Moltijoe
Copy link
Contributor Author

Moltijoe commented Jan 7, 2023

wdym with #17276 security can now fire lethal execution syringes at people.. which is kind of the logic of #17276 which you merged..?

A sec officer needs authorization from either the warden or HoS to take something from the armoury. Before, a medical doctor could just grab the syringe gun from their locker and no one would notice.

If the lethal injection syringe gun is as much of a problem as people say it will be, it could be given the piercing syringe treatment, where it cant fit into a syringe gun.

Also, i didnt expect jamie to spend more time on that PR, so I made this one.

@Moltijoe
Copy link
Contributor Author

Moltijoe commented Jan 7, 2023

Another idea would removing it from maps entirely and adding it to weapons research

@Hopekz
Copy link
Contributor

Hopekz commented Jan 7, 2023

A sec officer needs authorization from either the warden or HoS to take something from the armoury. Before, a medical doctor could just grab the syringe gun from their locker and no one would notice.

Yes that is what changed in #17276 , correct.

If the lethal injection syringe gun is as much of a problem as people say it will be, it could be given the piercing syringe treatment, where it cant fit into a syringe gun.

No idea; irrelevant to your PR imo.

Also, i didnt expect jamie to spend more time on that PR, so I made this one.

So you merged a PR in to production that you thought was unfinished?

@Moltijoe
Copy link
Contributor Author

Moltijoe commented Jan 7, 2023

No idea; irrelevant to your PR imo.

it is relevant, the problem is making sure syringe guns aren't being abused by valid hunters or griefers

So you merged a PR in to production that you thought was unfinished?

No, the PR was finished, and syringe guns shouldn't be so readily in the hands of regular players considering their power.
But that PR left it as being too restrictive

Also, the nature of this game is that things can constantly be changed, remember the 1 hit stun batons? they were removed when deemed shit. If after people play with it for a bit, it's actually bad rather than people just crying about not being able to validhunt with them, it can be reverted or some other such change.

@ReddicusDragon
Copy link
Contributor

molti if I merge this will jamie kill me

@Hopekz
Copy link
Contributor

Hopekz commented Jan 9, 2023

molti if I merge this will jamie kill me

Probably not

@ReddicusDragon
Copy link
Contributor

molti if I merge this will jamie kill me

Probably not

jamie DID make the PR to move it to armoury

@Hopekz
Copy link
Contributor

Hopekz commented Jan 9, 2023

jamie DID make the PR to move it to armoury

But you need to realize he has a horrible sense of game direction/balance so this PR is more of a course correction.

I'm actually surprised the previous PR got merged but it apparently was only because molti wanted to do something with it which in my opinion even after this PR gets merged puts us at a net negative vs just leaving the original PR unmerged until it was better.

@Moltijoe
Copy link
Contributor Author

Moltijoe commented Jan 9, 2023

I'm actually surprised the previous PR got merged but it apparently was only because molti wanted to do something with it which in my opinion even after this PR gets merged puts us at a net negative vs just leaving the original PR unmerged until it was better.

I dont care too much about public relations if it means something actually happens. Constant fuffing about of "oh, well it could be better this way" never gives any idea ground to actually be tested.

@ReddicusDragon ReddicusDragon merged commit 42ebb26 into yogstation13:master Jan 10, 2023
Yogbot-13 added a commit that referenced this pull request Jan 10, 2023
@Moltijoe Moltijoe mentioned this pull request Jan 12, 2023
@Moltijoe Moltijoe deleted the CMOsyringegun branch January 13, 2023 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This adds new content to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants