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

Slimes are no longer ghost roles #601

Merged

Conversation

RealIHaveANameOfficial
Copy link
Contributor

About the PR

Removed the ghost role components from MobAdultSlimes

Why / Balance

Reagent slimes being ghost roles is problematic, and I don't think there are any other situations where slimes are/need to be a ghost role here, so.

  • This PR does not require an ingame showcase

Breaking changes

Could affect other slime ghost roles but I don't think that will be an issue.

Changelog

🆑

  • remove: Slimes can no longer be taken over as a ghost role.

-This affects all ghost role slimes but is primarily geared to stop reagent slimes from becoming ghost roles.
makeSentient: true
name: ghost-role-information-slimes-name
description: ghost-role-information-slimes-description
- type: Speech
Copy link
Contributor

Choose a reason for hiding this comment

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

Dont remove speech please

@@ -111,15 +111,6 @@
successChance: 0.5
interactSuccessString: petting-success-slimes
interactFailureString: petting-failure-generic
- type: ReplacementAccent
accent: slimes
Copy link
Contributor

Choose a reason for hiding this comment

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

Dont remove lines, uncomment then add a note # Frontier so we can keep track of changes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you, bit silly of me to simply delete it outright, in hindsight.

@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Nov 19, 2023
Copy link
Contributor

@dvir001 dvir001 left a comment

Choose a reason for hiding this comment

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

Looking good.

@Cheackraze Cheackraze merged commit db8887a into new-frontiers-14:master Nov 22, 2023
10 checks passed
FrontierATC added a commit that referenced this pull request Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants