Skip to content

Ares' KeepAlive adds global tags - #2346

Open
Fly-Star-him wants to merge 7 commits into
Phobos-developers:developfrom
Fly-Star-him:keepaliveEX
Open

Ares' KeepAlive adds global tags#2346
Fly-Star-him wants to merge 7 commits into
Phobos-developers:developfrom
Fly-Star-him:keepaliveEX

Conversation

@Fly-Star-him

@Fly-Star-him Fly-Star-him commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Ares has hardcoded the default value of KeepAlive for buildings with Insignificant=no and DontScore=no as yes, while for other types of technologies, it is set to no. Now, four global tags have been added for customization.

Ares 将Insignificant=noDontScore=no的建筑的KeepAlive默认值设为yes,其他科技类型设为no。现添加了四个全局标签用于自定义设置。


In rulesmd.ini:

[General]
KeepAlive.SupportInfantrys=false    ; boolean
KeepAlive.SupportVehicles=false     ; boolean
KeepAlive.SupportAircrafts=false    ; boolean
KeepAlive.SupportBuildings=true    ; boolean

@phoboscn-bot

Copy link
Copy Markdown

To Chinese users:
This pull request has been mentioned on Phobos CN. There might be relevant details there:

致中文用户:
此拉取请求已在 Phobos CN 上被提及。那里可能有相关详细信息:

https://www.phoboscn.top/t/topic/716/1

@github-actions

github-actions Bot commented Aug 16, 2026

Copy link
Copy Markdown

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@TaranDahl

Copy link
Copy Markdown
Contributor

I think the original automatic value is already smart enough.
What are the specific use cases for this enhancement?

@Fly-Star-him

Fly-Star-him commented Aug 16, 2026

Copy link
Copy Markdown
Contributor Author

I think the original automatic value is already smart enough. What are the specific use cases for this enhancement?

You no longer need to frantically add KeepAlive=yes for each unit in the map file to achieve specific effects. Now, you only need to modify a maximum of four global tags.

@DeathFishAtEase DeathFishAtEase added the ⚙️T1 T1 maintainer review is sufficient label Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs testing ⚙️T1 T1 maintainer review is sufficient

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants