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

Added Yip Yip the kobold bartender #26265

Closed
wants to merge 1 commit into from

Conversation

PolterTzi
Copy link
Contributor

About the PR

Adds a kobold equivalent for Pun Pun.

Why / Balance

WEH

Media

image
image

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

🆑

  • add: A kobold might show up in Pun Pun's place.

@github-actions github-actions bot added the Changes: No C# Changes: Requires no C# knowledge to review or fix this item. label Mar 19, 2024
@Simyon264
Copy link
Contributor

FYI: There is a freeze on new pets.

New pets. While fluff is nice we have enough cargo-obtainable fluff animals and department mascots, to add new animals they must have something interesting like new AI behaviors or spawn in proc gen maps.

See #8524 for more info. This PR may be closed any time by a maintaner.

@Errant-4
Copy link
Member

At this point it might make sense to create a second parent for the bartender animals to inherit the common components without having to redefine them every time

@VasilisThePikachu VasilisThePikachu added the S: Frozen Status: Affects a game area that's frozen. Check "Current Freezes" in the Issues tab. label Mar 19, 2024
@VasilisThePikachu
Copy link
Member

FYI: There is a freeze on new pets.

New pets. While fluff is nice we have enough cargo-obtainable fluff animals and department mascots, to add new animals they must have something interesting like new AI behaviors or spawn in proc gen maps.

See #8524 for more info. This PR may be closed any time by a maintaner.

It wont be closed but not merged any time soon.

@PolterTzi
Copy link
Contributor Author

Ah, darn.
Oh well.

Copy link
Member

@VasilisThePikachu VasilisThePikachu left a comment

Choose a reason for hiding this comment

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

Though I will leave one review

@@ -539,7 +539,7 @@
- MobSlimesPet

- type: entity
name: Pun Pun Spawner
name: Bartender Animal Spawner
id: SpawnMobMonkeyPunpun
Copy link
Member

@VasilisThePikachu VasilisThePikachu Mar 19, 2024

Choose a reason for hiding this comment

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

I would suggest you rename the id as well and add a migration to change the old id to the new one

https://github.com/space-wizards/space-station-14/blob/master/Resources/migration.yml

@TheShuEd
Copy link
Member

I think it's fine and doesn't really fall under the freeze, since it's not adding a new ghost role, but randomizing the appearance of an existing one.

@VasilisThePikachu
Copy link
Member

I think it's fine and doesn't really fall under the freeze, since it's not adding a new ghost role, but randomizing the appearance of an existing one.

New pets. While fluff is nice we have enough cargo-obtainable fluff animals and department mascots, to add new animals they must have something interesting like new AI behaviors or spawn in proc gen maps.

I would call this a new department mascot. And it does not add anything interesting

@Carmell0
Copy link

Suggestions:
The name “nup nup” or “nip nip” would be funnier
Splash text saying “engage in a rivalry with pun pun”
Sometimes pun and yip spawn together. Sometimes both are at the bar, or maybe make it so yip spawns in the maints bar.

@CatMagic1
Copy link

I would argue that this is cool and yall are using TOO much logic to decide on funny lizard bartender

@UbaserB
Copy link
Member

UbaserB commented Mar 19, 2024

Suggestions:
The name “nup nup” or “nip nip” would be funnier
Splash text saying “engage in a rivalry with pun pun”
Sometimes pun and yip spawn together. Sometimes both are at the bar, or maybe make it so yip spawns in the maints bar.

i agree with this

@Hmeister-real
Copy link
Contributor

no explanation, just closed? This is a sad day

@VasilisThePikachu
Copy link
Member

no explanation, just closed? This is a sad day

they got repo banned

@Hmeister-real
Copy link
Contributor

no explanation, just closed? This is a sad day

they got repo banned

oh... what for?

@FairlySadPanda
Copy link
Contributor

ok hear me out
image

@deltanedas
Copy link
Contributor

ok hear me out
image

april 1 pun pun replacement

@Xman0613
Copy link

no explanation, just closed? This is a sad day

they got repo banned

wait why did they get repo banned?

@space-wizards space-wizards locked as resolved and limited conversation to collaborators Mar 20, 2024
@Jezithyr
Copy link
Contributor

Jezithyr commented Mar 20, 2024

Reopened since this author's repoban was in error. (The ban was made due to a misunderstood reference to the stonewall riots in an april fools PR)

@Jezithyr Jezithyr reopened this Mar 20, 2024
@space-wizards space-wizards unlocked this conversation Mar 20, 2024
@gitjubx
Copy link

gitjubx commented Mar 20, 2024

no explanation, just closed? This is a sad day

they got repo banned

wait why did they get repo banned?

he censort the LGBTQ capes and made a joke about it beeing funny to censoring it or such...then wiggled himself out so like punch then apologize and everything is fine!

@PolterTzi
Copy link
Contributor Author

no explanation, just closed? This is a sad day

they got repo banned

wait why did they get repo banned?

he censort the LGBTQ capes and made a joke about it beeing funny to censoring it or such...then wiggled himself out so like punch then apologize and everything is fine!

I censored them in the media preview of the april fools PR that was meant to add them to riot gear (since "pride was a riot" and all that).
I very much did not intend to censor them in game.
The censorship was admittedly not a good idea and that's on me.

@metalgearsloth
Copy link
Contributor

#25982 is a better approach to this.

@PolterTzi
Copy link
Contributor Author

I don't really agree that it's strictly better, but there's little point in me arguing when it's still frozen anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item. S: Frozen Status: Affects a game area that's frozen. Check "Current Freezes" in the Issues tab.
Projects
None yet
Development

Successfully merging this pull request may close these issues.