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

Disallow some guns from turrets based on action #72789

Merged
merged 2 commits into from Apr 9, 2024

Conversation

tenmillimaster
Copy link
Member

@tenmillimaster tenmillimaster commented Apr 2, 2024

Summary

None

Purpose of change

We are extremely permissive in which firearms we allow to be stuck in a turret mount (what even is a turret mount??). If we consider that the automated version of this is essentially a CROWS, many of these firearms don't make sense, either because they cannot be actuated by the turret controls (Slam fire, single-action revolvers, or lever actions), or because the firearms do not function when braced (inertia actions).

Describe the solution

Add "NO_TURRET" to relevant firearms.
Very generously assuming that sticking a tab or something to pull a pistol slide or work a pump slide.

Describe alternatives you've considered

Adding "NO_TURRET" to many, many, many more firearms.
Expanding this out to pump actions.
Getting gud at C++ and making turrets check if there's a turret mount(flag? mod slot?) on the firearm in question before allowing them to be mounted.

Testing

Loaded game, got debug hammer space, tried to install no-turreted firearms.

Additional context

None.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves labels Apr 2, 2024
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Apr 2, 2024
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 2, 2024
@Fris0uman Fris0uman merged commit 03e82ef into CleverRaven:master Apr 9, 2024
22 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants