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

Current Freezes and PR restrictions #8524

Open
wrexbe opened this issue May 29, 2022 · 21 comments
Open

Current Freezes and PR restrictions #8524

wrexbe opened this issue May 29, 2022 · 21 comments
Labels
Holy Shit This is real important!!

Comments

@wrexbe
Copy link
Contributor

wrexbe commented May 29, 2022

This is a list of things that are "frozen" or "restricted". See below for what each term means.

Freezes

PRs involving any of these things may be closed at maintainer discretion until the freeze is resolved.
If the freeze is due to an ongoing rework then the rework PR will take precedent and your PR may require reworking prior to merge.
As usual your PR is subject to not being merged into the repository, especially if maintainer approval is not sought beforehand.

  • New songs (lobby and jukebox) -
    image
    • Any new lobby songs will need to wait until a streaming system is implemented for them to be merged.
    • This includes jukebox music
  • Guns - Seek maintainer approval beforehand. (Ping @metalgearsloth and @DrSmugleaf)
  • Clothing variants for mobs - Inconsistently maintaining a lot of clothing sprite variants for species/mobs is not feasible and could have big long-term consequences. Variants will not be accepted until we support displacement masks (unless the variants have a good reason to be or a maintainer approved them). This freeze wont apply for maintaining current variants (like monkey suits).
    • Certain species-specific sprites are fine if they literally cant be displacement mapped.
    • It is fine for existing round-start species.
  • New pet clothing - These need to use displacement maps which have yet to be implemented. Gas tank sprites are still allowed.
  • 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.
  • Roundstart clothing - Wait for loadouts and don't make someone else have to update your yaml.
  • Wizard - Needs a design doc. (@keronshb is currently working on this.)
  • Magic/Spells/Spellbook - Currently refactoring & adding new ones (@keronshb is currently working on this.)

Restrictions

PRs surrounding any items in these list will be heavily scrutinized or require a design doc, taking longer to be reviewed.
Consult a maintainer when making a PR involving these things.

  • Antagonists - We have had code quality issues with these and the code will be scrutinised much more heavily than normal. (Requires a design doc before being implemented)
  • Roundstart / player species such as lizards / humans - A) These need design docs. These will be scrutinized for quality. B) There's existing species in 13 we may want to port instead C) the maintenance burden is enormous and not feasible for a lot of species.
  • Anything touching medical (reagents, entities, etc) due to woundmed coming - Any changes may not be re-implemented in the initial woundmed PR, but can be readded later by each PR author.
  • Recipes - More of a PSA that microwave recipes are subject to potential deletion in future pending chef refactor.
  • EMAG INTERACTIONS - we have too many and need to cut down. Needs a design doc.
  • Item Balance - These will be scrutinized heavily.
@wrexbe wrexbe pinned this issue May 29, 2022
@metalgearsloth

This comment was marked as resolved.

@Zumorica

This comment was marked as outdated.

@ZeroDayDaemon

This comment was marked as outdated.

@metalgearsloth
Copy link
Contributor

metalgearsloth commented Sep 7, 2022

Antags - We haven't actually had a meeting since I proposed it so it's tentatively here.
Reason is people work on a new antag (gameplay for one person occasionally) instead of job content (gameplay actively in every round for several players) so we continue our spiral of having no job content for departments like science. Fixing an existing antag is okay.

@Injazz
Copy link
Contributor

Injazz commented Nov 19, 2022

xenosci was merged so consider lifting freeze from research

@metalgearsloth

This comment was marked as outdated.

@juliangiebel

This comment was marked as outdated.

@EmoGarbage404

This comment was marked as outdated.

@keronshb
Copy link
Contributor

Restoring the antag freeze.

@deltanedas

This comment was marked as outdated.

@metalgearsloth
Copy link
Contributor

does borging count as species i wanna try it

We're discussing freezes more in discord but no. Borgs need to use body system though.

@deltanedas

This comment was marked as outdated.

@EmoGarbage404

This comment was marked as outdated.

@AJCM-git AJCM-git mentioned this issue May 7, 2023
1 task
@AjexRose

This comment was marked as outdated.

@EmoGarbage404

This comment was marked as outdated.

@moonheart08

This comment was marked as outdated.

@keronshb
Copy link
Contributor

Added Wizard since it still would require Mind Entities and the highest scrutiny.

@0x6273 0x6273 mentioned this issue Jun 25, 2023
1 task
@mirrorcult mirrorcult added the Holy Shit This is real important!! label Aug 25, 2023
@lzk228 lzk228 mentioned this issue Sep 10, 2023
1 task
@Boaz1111
Copy link
Contributor

Regarding the freeze on weapons, would a PR making certain weapons(musket, etc.) obtainable via a research be frozen?

@keronshb
Copy link
Contributor

keronshb commented Jan 5, 2024

Added item balance because we're not at a place where we have consistency internally.

@keronshb
Copy link
Contributor

Readded Wizard and Magic to freezes

@Partmedia Partmedia mentioned this issue Feb 14, 2024
1 task
This was referenced Mar 4, 2024
@UbaserB UbaserB mentioned this issue Mar 12, 2024
1 task
@DrSmugleaf DrSmugleaf changed the title Current Freezes Current Freezes and PR restrictions Apr 12, 2024
@VasilisThePikachu VasilisThePikachu unpinned this issue Apr 18, 2024
@VasilisThePikachu VasilisThePikachu pinned this issue Apr 18, 2024
@VasilisThePikachu
Copy link
Member

I fat-fingered remove, don't mind me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Holy Shit This is real important!!
Projects
None yet
Development

No branches or pull requests