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

Some Map Editor Actor cleanup #15987

Merged
merged 5 commits into from Jan 2, 2019

Conversation

Projects
None yet
4 participants
@MustaphaTR
Copy link
Member

MustaphaTR commented Jan 1, 2019

Removes Aircraft Husks from map editor, they are no use to be preplaced, they are gonna explode instantly anyway. Closes #15562.

Removes D2k duplicate actors from map editor. They are no different from normal ones except cost which don't matter when they are preplaced.

Removes aircraft with AttackBomber trait from map editor. Such units are meant to be spawned with AirstrikePower. Otherwise they crash the game.

Also noticed C17 and A10 had Buildable traits leftover and removed them too while in it.

@pchote pchote added this to the Next Release milestone Jan 1, 2019

@pchote

This comment has been minimized.

Copy link
Member

pchote commented Jan 1, 2019

This fits well with the rest of the editor changes, so adding to the milestone.

@pchote

pchote approved these changes Jan 1, 2019

@pchote pchote added the PR: Needs +2 label Jan 1, 2019

@matjaeck
Copy link
Contributor

matjaeck left a comment

LGTM

@reaperrr reaperrr merged commit baac3f3 into OpenRA:bleed Jan 2, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MustaphaTR MustaphaTR deleted the MustaphaTR:editor-actor-cleanup branch Jan 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment