Skip to content
This repository was archived by the owner on Apr 14, 2020. It is now read-only.

Vanilla Vehicles Expanded Patches #1171

Merged
3 commits merged into from
Apr 7, 2020
Merged

Conversation

Tostov
Copy link
Contributor

@Tostov Tostov commented Apr 1, 2020

Additions

Added patches for Vanilla Vehicles Expanded, for races and bodydef

Reasoning

Melee is roughly centipede level for the "strongest" vehicles, scaling down for other ones. They are cars after all, and although it's probably a little over scaled, I don't think this is either a serious mod, or a serious concern. Also cars should be fairly dangerous if they run into a human, even if not at full speed.

Testing

Check tests you have performed:

  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded

@N7Huntsman N7Huntsman changed the base branch from zhrocks11Dev to master April 1, 2020 20:35
MMMiracles
MMMiracles previously approved these changes Apr 2, 2020
Copy link

@MMMiracles MMMiracles left a comment

Choose a reason for hiding this comment

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

We can argue the delicate balance of man vs car when it comes to the authenticity CE strides to add but after testing I can confirm that it has no issues in the error logging. I can also confirm that you can technically win a fist fight against a car if you manage to punch out the tires fast enough.

@N7Huntsman N7Huntsman requested a review from a team April 4, 2020 05:40
@Tostov Tostov requested a review from a team as a code owner April 4, 2020 10:00
<value>
<tools>
<li Class="CombatExtended.ToolCE">
<label>head</label>
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be swapped to something else, given that the vehicles don't have "heads". Something like grille? body?

Copy link
Contributor Author

@Tostov Tostov Apr 6, 2020

Choose a reason for hiding this comment

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

Ahh, nice catch. Should be fixed now, consistent with base mod.

@N7Huntsman
Copy link
Contributor

@MMMiracles If you think it's good to go, please put an Approval on it under reviews, and I'll get it merged.

@ghost ghost merged commit ce55200 into CombatExtendedRWMod:master Apr 7, 2020
@Tostov Tostov deleted the VAEAprilFools branch November 9, 2020 17:37
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants