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

Remove bubble car spawns #38050

Merged
merged 1 commit into from
Feb 15, 2020
Merged

Conversation

anothersimulacrum
Copy link
Member

@anothersimulacrum anothersimulacrum commented Feb 15, 2020

Summary

SUMMARY: None

Purpose of change

Minireactor powered vehicles haven't fit in canon fore a while, so they shouldn't be spawning.

Describe the solution

Remove bubble car spawns from core, and stick them in Aftershock

Describe alternatives you've considered

Just removing them.

Testing

tests/cata_test --mods="aftershock" '~*' runs without errors (loading the data for the mod).
Running the tests have no failures. (Attempting to run the tests with aftershock leads to unrelated vehicle weight failures).

Minireactor powered vehicles haven't fit in canon fore a while, so they
shouldn't be spawning.
Have them spawn in Aftershock instead.
@I-am-Erk I-am-Erk added [JSON] Changes (can be) made in JSON Mods: Aftershock Anything to do with the Aftershock mod Vehicles Vehicles, parts, mechanics & interactions labels Feb 15, 2020
@I-am-Erk I-am-Erk merged commit 26b2614 into CleverRaven:master Feb 15, 2020
@anothersimulacrum anothersimulacrum deleted the bubble-shock branch February 15, 2020 22:10
anothersimulacrum added a commit to anothersimulacrum/Cataclysm-DDA that referenced this pull request Jun 19, 2020
Atomic cars haven't been canon in vanilla for a while (CleverRaven#38050), and this
appears to have been errorneously added in CleverRaven#40706
anothersimulacrum added a commit to anothersimulacrum/Cataclysm-DDA that referenced this pull request Jun 19, 2020
Atomic cars haven't been canon in vanilla for a while (CleverRaven#38050), and this
appears to have been erroneously added in CleverRaven#40706
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON Mods: Aftershock Anything to do with the Aftershock mod Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants