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

Skill level description #79761

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

Conversation

GuardianDll
Copy link
Member

Summary

None

Purpose of change

Long time ago someone proposed to move what we have in https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/design-balance-lore/GAME_BALANCE.md#skill-system-scaling to a game in some way

Describe the solution

Implement two fields, that allow to define a message, that would allow the player to clearly understand at what point they are with this skill

Describe alternatives you've considered

Not doing it?

Testing

image

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. [C++] Changes (can be) made in C++. Previously named `Code` [Markdown] Markdown issues and PRs json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Feb 20, 2025
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON and removed BasicBuildPassed This PR builds correctly, label assigned by github actions labels Feb 21, 2025
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 21, 2025
@AutPhoenix
Copy link

Change "you likely have a PhD" to "You would probably have a PhD if the universities weren't full of zombies" perhaps.

"level": 5,
"description": "You can easily craft useful, if junky weapons and armor, but nothing too complex."
},
{ "level": 7, "description": "You're can craft even complex objects without fail." },
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{ "level": 7, "description": "You're can craft even complex objects without fail." },
{ "level": 7, "description": "You can craft even complex objects without fail." },

"level": 3,
"description": "You've learned enough to avoid overswinging and falling off balance if you miss."
},
{ "level": 5, "description": "Whether self-taught or trained you've learned quite a lot about fighting." },
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{ "level": 5, "description": "Whether self-taught or trained you've learned quite a lot about fighting." },
{ "level": 5, "description": "Whether self-taught or trained, you've learned quite a lot about fighting." },

{ "level": 1, "description": "You can usually get a decent cut when chopping." },
{ "level": 3, "description": "You can reliably keep good edge alignment when slicing." },
{ "level": 5, "description": "You're a fairly skilled swordsman." },
{ "level": 9, "description": "With your skill, In another life you could duel a samurai." }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{ "level": 9, "description": "With your skill, In another life you could duel a samurai." }
{ "level": 9, "description": "With your skill, in another life you could duel a samurai." }

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 [C++] Changes (can be) made in C++. Previously named `Code` <Documentation> Design documents, internal info, guides and help. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions [Markdown] Markdown issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants