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

Artillery shells don't use casings #3543

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Conversation

N7Huntsman
Copy link
Contributor

@N7Huntsman N7Huntsman commented Nov 11, 2024

Changes

A very minor one. Makes the 155mm and 105mm howitzer shells no longer drop casings.

Reasoning

Realism. 105mm and 155mm shells being fired from artillery guns don't use fixed-cased projectiles, instead using bag charges. There are fixed-case 105mm shells that I've found, but we currently don't have anything that would use those.

Alternatives

  • Leave it as-is.
  • Create separate variants of the 105 and 155 projectiles that do spit out casings when fired, in case we end up needing them.

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony (specify how long)

@N7Huntsman N7Huntsman requested review from a team as code owners November 11, 2024 20:00
Copy link

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

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Nov 11, 2024
@N7Huntsman N7Huntsman merged commit a3a75a3 into Development Nov 11, 2024
3 checks passed
@N7Huntsman N7Huntsman deleted the ArtilleryCasings branch November 11, 2024 22:44
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.

2 participants