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

Adds new profession: Kitted Survivor #35650

Merged
merged 6 commits into from Nov 29, 2019
Merged

Adds new profession: Kitted Survivor #35650

merged 6 commits into from Nov 29, 2019

Conversation

ghost
Copy link

@ghost ghost commented Nov 22, 2019

Summary

SUMMARY: Content "Added new Profession 'Kitted Survivor'"

Purpose of change

Describe the solution

Allows players to start with a survival kit, which might aid them.

Describe alternatives you've considered

Not adding this profession.

Testing

Additional context

Xenomorph-III added 2 commits November 21, 2019 22:49
Costs 1 point. Starts with worse clothing than the ordinary Survivor but has a full survival kit which can aid them.

Inspired by Cataclysm++'s "Canned Survivor", who starts with a decked out "Survival In A Can" kit.
"type": "profession",
"ident": "kit_survivor",
"name": "Kitted Survivor",
"description": "At the local mall, you saw a sign advertising a discount on survival kits. You bought one, more for show than for actual use. Now, it's all you have."
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
"description": "At the local mall, you saw a sign advertising a discount on survival kits. You bought one, more for show than for actual use. Now, it's all you have."
"description": "At the local mall, you saw a sign advertising a discount on survival kits. You bought one, more for show than for actual use. Now, it's all you have.",

Copy link
Author

@ghost ghost Nov 22, 2019

Choose a reason for hiding this comment

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

Oh - how do I lint my JSON?

And, if you have any other description ideas, feel free to suggest them.

Copy link
Contributor

Choose a reason for hiding this comment

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

Lint the whole file contents with a http://dev.narc.ro/cataclysm/format.html

Copy link
Member

@anothersimulacrum anothersimulacrum 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 lint your JSON.

@AMurkin
Copy link
Contributor

AMurkin commented Nov 22, 2019

And fix the summary.
SUMMARY: Content "Added new Profession 'Kitted Survivor'"

Missing Comma

Co-Authored-By: anothersimulacrum <anothersimulacrum@gmail.com>
data/json/professions.json Outdated Show resolved Hide resolved
Double spaces.

Co-Authored-By: Alexey Mostovoy <1931904+AMurkin@users.noreply.github.com>
data/json/professions.json Outdated Show resolved Hide resolved
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies labels Nov 22, 2019
Removed extra comma

Co-Authored-By: Alexey Mostovoy <1931904+AMurkin@users.noreply.github.com>
@ZhilkinSerg ZhilkinSerg merged commit de45acc into CleverRaven:master Nov 29, 2019
@ghost ghost deleted the kitted_survivor branch February 15, 2020 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants