Skip to content

Conversation

@PatrikLundell
Copy link
Contributor

@PatrikLundell PatrikLundell commented Nov 24, 2023

Summary

None

Purpose of change

Add weakpoints and weakpoint proficiencies to slimes.

Describe the solution

Added the required entries.

Describe alternatives you've considered

It could be debated if fighting your own slimelings should gain you these proficiencies, but on the other hand, being a slime should really give you a lot of knowledge in itself. Also, you might potentially encounter a hostile slime mutant with its slimelings.

Testing

  • Debug spawned a slime and fought it (and its "offspring"), seeing weakpoint messages triggering.
  • Checked that the basic proficiency was added, and it wasn't... It turns out I'd somehow missed the "basic" part of the name, and the game didn't detect that the stunted proficiency doesn't exist.
  • Fixed the issue, retested, and now received the proficiency entry.
  • Debug mastered the basic proficiency, spawned a new slime, fought it, and verified that the advanced proficiency appeared as well (better check when an erroneous name for the basic one went AWOL).

Additional context

The organ weakpoint was based on a lab snippet I've seen, which I assume was referring to slimes.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Nov 24, 2023
@github-actions
Copy link
Contributor

Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details.

Click to expand
  • an organ inside the slime, causing the slime to freeze up briefly as it reconstitutes it

This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to tools/spell_checker/dictionary.txt so they will not trigger an alert next time.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Nov 24, 2023
@Maleclypse Maleclypse merged commit ddbc660 into CleverRaven:master Nov 24, 2023
@PatrikLundell PatrikLundell deleted the slime_wp_prof branch November 25, 2023 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants