Skip to content

Ares' KeepAlive adds global tags - #2346

Merged
Coronia merged 9 commits into
Phobos-developers:developfrom
Fly-Star-him:keepaliveEX
Aug 20, 2026
Merged

Ares' KeepAlive adds global tags#2346
Coronia merged 9 commits into
Phobos-developers:developfrom
Fly-Star-him:keepaliveEX

Conversation

@Fly-Star-him

@Fly-Star-him Fly-Star-him commented Aug 16, 2026 •
edited by Coronia
Loading

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, five global tags have been added for customization.

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

In rulesmd.ini:

[General]
KeepAlive.Buildings=true      ; boolean
KeepAlive.Defenses=true       ; boolean
KeepAlive.Infantry=false      ; boolean
KeepAlive.Units=false         ; boolean
KeepAlive.Aircraft=false      ; boolean

@phoboscn-bot

phoboscn-bot commented Aug 16, 2026

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 •
edited
Loading

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

TaranDahl commented Aug 16, 2026

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 •
edited
Loading

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
Coronia
Coronia approved these changes Aug 20, 2026
View reviewed changes

@Coronia Coronia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

modified the tag names a bit to make it inline with other tags that works for different abstract types

@Coronia
Coronia merged commit d6c0bc5 into Phobos-developers:develop Aug 20, 2026
7 checks passed
DeathFishAtEase added a commit to DeathFishAtEase/Phobos that referenced this pull request Aug 20, 2026
@DeathFishAtEase
[cn] update
8dcbcba 
Phobos-developers#2346
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Coronia Coronia Coronia approved these changes

Assignees

No one assigned

Labels

⚙️T1 T1 maintainer review is sufficient Tested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Fly-Star-him @phoboscn-bot @TaranDahl @Coronia @DeathFishAtEase