-
Notifications
You must be signed in to change notification settings - Fork 53
Added Costumes/Clothing and Fixes #209
base: master
Are you sure you want to change the base?
Conversation
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.
You need to install EditorConfig
Resources/Prototypes/DeltaV/Entities/Clothing/OuterClothing/vests.yml
Outdated
Show resolved
Hide resolved
Resources/Prototypes/Catalog/VendingMachines/Inventories/salvage.yml
Outdated
Show resolved
Hide resolved
Resources/Textures/DeltaV/Clothing/OuterClothing/Coats/skeletoncostume.rsi/meta.json
Outdated
Show resolved
Hide resolved
Resources/Textures/DeltaV/Clothing/OuterClothing/Vests/amonggrande.rsi/meta.json
Outdated
Show resolved
Hide resolved
…sts.yml Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
…ge.yml Co-authored-by: DEATHB4DEFEAT <[email protected]>
…ande.rsi/meta.json Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
…ncostume.rsi/meta.json Co-authored-by: DEATHB4DEFEAT <[email protected]>
Is that required to make this PR work? |
No it's just a good extension to have, fixes some formatting mistakes to follow project standards |
You need to fix your changelogs |
Resources/Prototypes/Catalog/VendingMachines/Inventories/salvage.yml
Outdated
Show resolved
Hide resolved
@@ -650,7 +650,7 @@ | |||
- type: entity | |||
parent: ClothingHeadBase | |||
id: ClothingHeadHatTacticalMaidHeadband | |||
name: tactical maid headband | |||
name: red headband |
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.
name: red headband | |
name: tactical maid headband |
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.
Would renaming it to red headband not be good? Tactical maid headband doesn't really make sense to me for what it looks like and what the description is.
@@ -1,7 +1,7 @@ | |||
{ | |||
"version": 1, | |||
"license": "CC-BY-SA-3.0", | |||
"copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/4f6190e2895e09116663ef282d3ce1d8b35c032e", | |||
"copyright": "Made by @ps3moira#9488", |
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.
No sprites were changed, is this correct?
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.
No that is not whoops
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.
File path is incorrect and contains a space.
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.
Move to DeltaV folder.
…ge.yml Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
About the PR
Media
Changelog
🆑