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

SilverCore research for the silver golem #94

Merged
merged 15 commits into from
Oct 19, 2023

Conversation

FluffiestFloof
Copy link
Contributor

@FluffiestFloof FluffiestFloof commented Oct 3, 2023

About the PR

Silver Golem not included.
Neither are the rest of the research items related to this one like QSI and HandTeleporter.
Includes all the golem sprites tho

Why / Balance

Technical details

Media

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

Breaking changes

Changelog

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

RSI Diff Bot; head commit d3dd2cf merging into 6e72475
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Nyanotrasen/Mobs/Golem/cult.rsi

State Old New Status
cultgolem_chest Added
cultgolem_head Added
cultgolem_l_arm Added
cultgolem_l_hand Added
cultgolem_l_leg Added
cultgolem_r_arm Added
cultgolem_r_hand Added
cultgolem_r_leg Added
full Added
lights Added

Resources/Textures/Nyanotrasen/Mobs/Golem/silver.rsi

State Old New Status
full Added
lights Added
silver_base Added

Resources/Textures/Nyanotrasen/Mobs/Golem/wood.rsi

State Old New Status
head Added
lights Added

Resources/Textures/Nyanotrasen/Objects/Devices/QSI.rsi

State Old New Status
icon Added

Edit: diff updated after d3dd2cf

@github-actions github-actions bot added Changes: Sprites Changes: YML Changes any yml files S: Merge Conflict Fix your PR! labels Oct 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: Fluffiest Floofers <[email protected]>
@github-actions github-actions bot removed the S: Merge Conflict Fix your PR! label Oct 3, 2023
@DEATHB4DEFEAT
Copy link
Member

> "no golem"
> look inside
> golem

:trollface:

Co-authored-by: DEATHB4DEFEAT <[email protected]>
Signed-off-by: Fluffiest Floofers <[email protected]>
@github-actions github-actions bot added the S: Merge Conflict Fix your PR! label Oct 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: Fluffiest Floofers <[email protected]>
@github-actions github-actions bot removed the S: Merge Conflict Fix your PR! label Oct 3, 2023
Copy link
Contributor

@Colin-Tel Colin-Tel left a comment

Choose a reason for hiding this comment

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

I love comments!

@DebugOk
Copy link
Contributor

DebugOk commented Oct 5, 2023

@DEATHB4DEFEAT Please re-review and change it if applicable

@DebugOk
Copy link
Contributor

DebugOk commented Oct 19, 2023

Considering it seems to work fine, and I dont see why we should keep this waiting any longer, I'll just merge it without waiting for death to re-review :clueless:

@DebugOk DebugOk merged commit 8adc0a3 into DeltaV-Station:master Oct 19, 2023
12 checks passed
@FluffiestFloof FluffiestFloof deleted the silvergolem branch October 19, 2023 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants