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.

Sign up for GitHub

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

Jump to bottom

Change Subs to use the Defend stance by default #18237

Merged
merged 2 commits into from Jun 19, 2020
Merged

Change Subs to use the Defend stance by default #18237

merged 2 commits into from Jun 19, 2020

Conversation

abcdefg30
Copy link
Member

This has been asked for several times already. As is you have to change the Subs stance, otherwise you will lose them when not paying constant attention since other naval units will just pick them off when attack moving.

@abcdefg30 abcdefg30 added this to the Next Release milestone Jun 5, 2020
@abcdefg30
Copy link
Member Author

We also might want to think about making the AI stance to AttackAnything by default. Missions already define overrides to do that.

@deleted-user-1
Copy link
Contributor

👍 to the changing the default stance to Defend but is it expected that subs will autotarget enemy shipyards / sub pens (shouldn't they only do in when in AttackAnything stance?

@abcdefg30
Copy link
Member Author

I'll fix that and make the AI use AttackAnything then.

@KOYK
Copy link
Contributor

KOYK commented Jun 14, 2020

Finally!

@abcdefg30
Copy link
Member Author

Updated.

@reaperrr reaperrr merged commit 318c4e3 into OpenRA:bleed Jun 19, 2020
@reaperrr
Copy link
Contributor

https://github.com/OpenRA/OpenRA/wiki/Changelog-(bleed)/_compare/40dbc851bb1fad21eb2b3c08d30e3bda120cd7fe...33278e30ffc3955e60b9ce1f1cdd0c6762a96955

@abcdefg30 abcdefg30 deleted the defendSubs branch June 19, 2020 15:48
@abcdefg30 abcdefg30 mentioned this pull request Jun 23, 2020
Merged
@abcdefg30 abcdefg30 mentioned this pull request Jul 24, 2020
Merged
@abcdefg30 abcdefg30 mentioned this pull request Oct 11, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reaperrr reaperrr reaperrr approved these changes

@Smittytron Smittytron Smittytron approved these changes

Assignees
No one assigned
Labels
PR: Needs +2
Projects
None yet
Milestone
Release 20210321
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@abcdefg30 @deleted-user-1 @KOYK @reaperrr @Smittytron