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

Disable nukes completely #245

Merged
merged 2 commits into from
Sep 7, 2023

Conversation

oof2win2
Copy link
Contributor

@oof2win2 oof2win2 commented Sep 6, 2023

This PR disables nukes completely by disabling the research for them (configgable), as they don't have much of real use besides griefers using them.

Copy link
Member

@Cooldude2606 Cooldude2606 left a comment

Choose a reason for hiding this comment

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

I think the default should be "enabled" because in the late game people use spidertrons with nukes. However, this feature is still useful for server that do not reach that stage of the game so this can still be accepted.

@oof2win2
Copy link
Contributor Author

oof2win2 commented Sep 7, 2023

Fixed those changes. With experience from our community, most people just abuse nukes overall since the blast radius is too large for spidertrons or rocket launchers.

Copy link
Member

@Cooldude2606 Cooldude2606 left a comment

Choose a reason for hiding this comment

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

I spoke with moderators within our community and the consensus was that the greatest benefit from disabling nukes is that they can no longer be used to create lag. However, artillary shells are a much cheeper, simpler, and more effective way to achieve the same result. Therefore, there is not alot to be gained from disabling them.

@Cooldude2606 Cooldude2606 merged commit 72153df into explosivegaming:dev Sep 7, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants