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

Removes Nar-Sie from portal storm event #17622

Merged
merged 12 commits into from
May 15, 2016
Merged

Removes Nar-Sie from portal storm event #17622

merged 12 commits into from
May 15, 2016

Conversation

Robustin
Copy link
Contributor

@Robustin Robustin commented May 12, 2016

Who the fuck is ever going to run an event that kills everyone and immediately ends the round? If I want to spawn Nar-Sie ill just use the game panel.

Conclusion: It takes me 5 seconds to summon a nar-sie, I don't need an event button to do that for me, but realistically no admin can pull off a portal storm without this event code and nobody would ever want to spawn Nar-Sie as an event. So, I'm trying to salvage this useless button from being something other than deadmin bait.

Now called "Portal Storm: Constructs" to satisfy certain supernerds.

@MMMiracles
Copy link
Contributor

someone actually thought that spawning nar-sie from a random event was a good idea

memes

@kilkun
Copy link
Contributor

kilkun commented May 12, 2016

it's for deep lore admin events tbh. 👎

@TehZombehz
Copy link
Contributor

I could have sworn that was removed ages ago. I guess it was stealthily retained for memes.

@Robustin
Copy link
Contributor Author

Ah the xxAlpha and Gunhog thumbsdown, the strongest guarantee that your PR is improving the game.

I knew it was an admin only event dipshit. There was a cult round where the cult got wiped out but I wanted to give the crew something thematic to fight near the end of the round. I considered this portal storm but LOL IT SPAWNS NAR SIE SO YOU CAN NEVER JUSTIFY TRIGGERING IT.

I'm trying to salvage this shit event so its not just another useless "DONT PRESS THIS" event cluttering up the menu. its either that or I can put it this code in a "removes shitty events that nobody would ever run" PR.

boss_types = list(/obj/singularity/narsie/large = 1)
hostile_types = list(/mob/living/simple_animal/hostile/construct/armored/hostile = 10,\
boss_types = list(/mob/living/simple_animal/hostile/construct/armored/hostile = 2)
hostile_types = list(/mob/living/simple_animal/hostile/construct/armored/hostile = 8,\
/mob/living/simple_animal/hostile/construct/wraith/hostile = 10)
Copy link
Contributor

Choose a reason for hiding this comment

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

You should add a few hostile artificers, like 3-4. They run from everything but will find and heal constructs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yea I forgot that their AI was actually pretty good

@Bawhoppen
Copy link
Contributor

wow robustin I literally did not think you could get more retarded 'WHY ARE THERE BUTTONS I CANT PRESS REMOVE'

@Robustin
Copy link
Contributor Author

Because I saw a utility in a portal storm event that summoned constructs but not Nar-Sie, is that so hard to understand? Apparently it is given how xxalpha was so overcome with rage at seeing his defunct code altered.

THIS BUTTON HAS NO PURPOSE, I AM GIVING IT PURPOSE, WHY IS THIS A DIFFICULT CONCEPT FOR SOME OF YOU?

@Archie700
Copy link
Contributor

So did people actually read the coding before commenting?

Also this is why you make changelogs to prevent splurge.

@Dorsisdwarf Dorsisdwarf added the Removal This was too fun, too fun! I'm turning this feature around label May 13, 2016
@Dorsisdwarf
Copy link
Contributor

Why the hell would you have an event that summons narsie, even an admin event. If you want Narsie, spawn narsie. This is a good PR because we can now cultbomb the station without

BIG TEXT HAS RISEN

@Archie700
Copy link
Contributor

Also a admin event of NAR'SIE is pretty much GG ROUND END FUCK YOU GUYS.

No admin who wants to be deadminned immediately would even press this in the middle of a round without very good reasoning and, as Forceful said, Nar'Sie can be spawned from the panel. So, the question is, WHY BOTHER HAVING THE CODE AT ALL IT'S REDUNDANT.

@Robustin
Copy link
Contributor Author

Thank god the sanity squad has arrived. This place will be the end of me.

boss_types = list(/obj/singularity/narsie/large = 1)
hostile_types = list(/mob/living/simple_animal/hostile/construct/armored/hostile = 10,\
/mob/living/simple_animal/hostile/construct/wraith/hostile = 10)
boss_types = list(/mob/living/simple_animal/hostile/construct/builder = 6)
Copy link
Contributor

Choose a reason for hiding this comment

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

Six artis and no harvestors?

Copy link
Contributor

Choose a reason for hiding this comment

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

Why would you need harvesters

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Harvesters are supposed to be unique to Nar-Sie, they don't really belong anywhere else.

Copy link
Contributor

Choose a reason for hiding this comment

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

The artificers wont spawn until the very end

Copy link
Contributor

Choose a reason for hiding this comment

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

Thought the boss was at the end of the wave, huh

Copy link
Contributor

Choose a reason for hiding this comment

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

Harvester hostile mobs are boring and weak anyway because they don't cast shit.

@ChangelingRain ChangelingRain merged commit 7d045c4 into tgstation:master May 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Removal This was too fun, too fun! I'm turning this feature around
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants