Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign up[Rdy] [CR] Skeletons overhaul #23982
Conversation
Night-Pryanik
reviewed
Jun 10, 2018
| "default_faction": "zombie", | ||
| "species": [ "ZOMBIE" ], | ||
| "diff": 12, | ||
| "size": "SMALL", | ||
| "hp": 12, | ||
| "speed": 120, | ||
| "speed": 100, |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
nexusmrsep
Jun 10, 2018
Author
Contributor
As stated in #23463 the overgrown bone carapace and reduced muscle mass would reduce speed.
This comment has been minimized.
This comment has been minimized.
|
Both descriptions are both really awkward and stilited, I like where you're going with it but it's difficult to follow. Can you come up with a few more examples? Dog: Skeleton: |
DracoGriffin
added
Game: Balance
[JSON]
Monsters
labels
Jun 11, 2018
This comment has been minimized.
This comment has been minimized.
|
Yeah, all suggestions are welcomed as I'm not a native speaker. Dog, skeletal: Variant 1: Variant 2: Skeleton: Variant 1: Variant 2 Feel free to discuss, vote for one of the variants or propose another. |
This comment has been minimized.
This comment has been minimized.
|
it's -> contraction of 'it is' |
ZhilkinSerg
assigned
ZhilkinSerg
and unassigned
ZhilkinSerg
Jun 13, 2018
nexusmrsep
changed the title
Skeletons overhaul part 1
[CR] Skeletons overhaul part 1
Jun 13, 2018
This comment has been minimized.
This comment has been minimized.
|
Honestly, after the variants, I think I like my examples more. Otherwise, variant 1 for canine, variant 2 for skeleton. |
This comment has been minimized.
This comment has been minimized.
I went for your variant for skeleton and my variant nr 1 for canine skeleton. |
nexusmrsep
changed the title
[CR] Skeletons overhaul part 1
[WiP] [CR] Skeletons overhaul
Jul 1, 2018
nexusmrsep
added some commits
Jul 1, 2018
This comment has been minimized.
This comment has been minimized.
|
Ok, so after adding mostly everything in the source issue #23463 I have some questions before I flag this PR as READY:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
|
Ok, as I am also not convinced to add anything more at the moment without more feedback from the gameplay, let's consider this PR done, and see if more tweaking needs come up with time. |
nexusmrsep
changed the title
[WiP] [CR] Skeletons overhaul
[Rdy] [CR] Skeletons overhaul
Jul 1, 2018
kevingranade
self-assigned this
Jul 4, 2018
This comment has been minimized.
This comment has been minimized.
|
I'm testing this right now, and just FYI: |
This comment has been minimized.
This comment has been minimized.
|
Bug: when skeleton armor negates damage, it doesn't print what the round bounces off of. |
nexusmrsep commentedJun 10, 2018
•
edited
Resolves #23463
This may be
part one ofthe skeleton overhaul proposed by @kevingranade .Things done in this pull request: