Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

Added Costumes/Clothing and Fixes #209

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

ps3moira
Copy link
Contributor

@ps3moira ps3moira commented Oct 4, 2023

About the PR

  • added among grande costume
  • added skeleton costume
  • swapped sprite salvage jumpsuit sprites
  • added more supplies to salvage vend
  • added clothing to clothesmate
  • removed clothing to clothesmate
  • added clothing to winterdrobe
  • added clothing to secdrobe
  • added cc armored winter coat to centdrobe
  • added hos's mantle +flask to their locker
  • added mystagogue's resprited cloak to their locker

Media
image

image

image

image

Screenshot 2023-10-04 050722

image

image

image

Changelog

🆑

  • add: Added Among Grande and Skeleton Costume
  • add: Added Costumes and Clothing to Vends
  • tweak: Salvage Jumpsuit Sprites
  • tweak: Added Salvage Jumpsuit and Gear to Salvage Vend

@github-actions github-actions bot added Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files labels Oct 4, 2023
Copy link
Member

@DEATHB4DEFEAT DEATHB4DEFEAT left a 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

@ps3moira
Copy link
Contributor Author

ps3moira commented Oct 5, 2023

You need to install EditorConfig

Is that required to make this PR work?

@github-actions github-actions bot added the Status: Needs Review Someone please review this label Oct 5, 2023
@DEATHB4DEFEAT
Copy link
Member

You need to install EditorConfig

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

@DEATHB4DEFEAT
Copy link
Member

  • add: Among Grande and Skeleton Costumes
  • add: Costumes and Clothing to Vends
  • tweak: Salvage Jumpsuit Sprites

Don't consider the entry type suffix (e.g. add) to be "part" of the sentence:
bad: - add: a new tool for engineers
good: - add: Added a new tool for engineers

You need to fix your changelogs

@@ -650,7 +650,7 @@
- type: entity
parent: ClothingHeadBase
id: ClothingHeadHatTacticalMaidHeadband
name: tactical maid headband
name: red headband
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name: red headband
name: tactical maid headband

Copy link
Contributor Author

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",
Copy link
Member

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?

Copy link
Contributor Author

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

Copy link
Member

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.

Copy link
Member

Choose a reason for hiding this comment

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

Move to DeltaV folder.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants