-
-
Notifications
You must be signed in to change notification settings - Fork 681
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
Fixes Offset Table #11872
base: master
Are you sure you want to change the base?
Fixes Offset Table #11872
Conversation
The smallest PR I've made to date. |
soul removal 🔑 |
pixel_x: | ||
pixel_y: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pixel_x: | |
pixel_y: | |
- pixel_x | |
- pixel_y |
This is only syntactic, but name: should be used for when we are defining something inside of it:
If we want to be more specific, we can write out the specific variables we want to ban.
/mob/dog
banned_variables:
- species # Don't var-edit species, use the subtypes
We can also explicitly create allowlists and denylists of values through allow
and deny
. For example, if we want to make sure we're not creating invalid bowls for animals, we can write:
/obj/bowl/dog:
banned_variables:
species:
# If we specify a species, it's gotta be a dog
allow: ["beagle", "corgi", "pomeranian"]
/obj/bowl/humans:
banned_variables:
species:
# We're civilized, we don't want to eat from the same bowl that's var-edited for animals
deny: ["cats", "dogs"]
About The Pull Request
Table in Science Security has been offset for like two months now. Time to fix it.
Why It's Good For The Game
Maybe finally BeeStation with get 100+ pop?
Testing Photographs and Procedure
Screenshots&Videos
Works!
Changelog
🆑
fix: (Meta) Fixed offset table in Science Security Outpost.
/:cl: