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

Armor durability apply damage fix for non-health damaging or non external violence damage #3366

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

CMDR-Bill-Doors
Copy link
Contributor

@CMDR-Bill-Doors CMDR-Bill-Doors commented Aug 23, 2024

Changes

Natural armor durability will not be reduced from damage that def harmshealth = false (firefoams) or is external violence (frostbite and SOS2 vacuum damage something)

Reasoning

-As we all know, firefoam shell is the best AT weapon in the universe, we should equip our armies with fire engines.

Alternatives

-Suffer.

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • Playtested a colony (a centipede vs 100 pirates, and a centipede vs firefoam shell)

@CMDR-Bill-Doors CMDR-Bill-Doors requested review from a team as code owners August 23, 2024 09:19
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-10523240417.zip

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Aug 23, 2024
Copy link
Contributor

@ViralReaction ViralReaction left a comment

Choose a reason for hiding this comment

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

Appears to be properly preventing firefoam grenades from stripping all armor. Tested on Militor and armor did not change.

@N7Huntsman N7Huntsman merged commit 7461caa into Development Aug 24, 2024
3 checks passed
@N7Huntsman N7Huntsman deleted the ArmorDurabilityApplyDamageFix branch August 24, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Download in Comments This PR has a zipfile download available.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants